API Suggestion


#1

It’s not intuitive to have two hashes on any one method call. Most of
the view helpers have a hash for method options followed by a hash for
html options. I find it to be problematic at best. I think it would be
best to have one hash and then add your html options as a hash entry.
For example:

<%= link_to “All Facilities”, {:action => ‘fac_log_all’}, {:target =>
‘_new’} %>

would become

<%= link_to “All Facilities”, :action => ‘fac_log_all’, :html =>
{:target => ‘_new’} %>

Or even better, any options that are not recognized by the method are
automatically treated as html options:

<%= link_to “All Facilities”, :action => ‘fac_log_all’, :target =>
‘_new’ %>

I’ve seen some examples done both ways, but whatever it is it should be
consistent.