El Monday 04 May 2009 19:57:33 dkindlund escribió:
> Yeah, I just figured that out, too. Actually, I have another idea:
>
> in app/views/active_scaffold_overrides/_list_record.html.erb:
> ...
> if defined?(tr_class).nil?
> tr_class = cycle("", "even-record even-record-")
> tr_class += "#{list_row_class(record)}" if
> respond_to? :list_row_class
> end
> ...
>
> That preserves the original "even-record" semantic but also adds a
> "even-record-class" attribute, allowing you to figure out when a row
> corresponds to "class" and is an "even-record".
> -- Darien
Ok, I can change it, but have you try to put tr.even-record.css_class in your
css?
>
> On May 4, 10:41 am, "Sergio Cambra .:: entreCables S.L. ::."
>
> <[email protected]> wrote:
> > Recover even-record class it's working in ActiveScaffold. You broke it
> > changing even-record to even-record-
> >
> > El Friday 01 May 2009 01:56:04 dkindlund escribió:
> > > After further investigation, it looks like there was a bug in the
> > > partial row rendering, which made it impossible to recover an "even or
> > > odd-specific" row class (tr class) after closing a sub nested table.
> > >
> > > I've been able to fix the problem and have provided a basic patch
> > > below. Just to a search for 'tr_class' in the code below, and you'll
> > > see all the changes from the master branch:
> > >
> > > vendor/plugins/active_scaffold/lib/active_scaffold/actions/list.rb:29
> > > --------------
> > > def row
> > > render :partial => 'list_record', :locals => {:record =>
> > > find_if_allowed(params[:id], :read), :tr_class => para
> > > ms[:tr_class]}
> > > end
> > > --------------
> > > app/views/active_scaffold_overrides/_list_record.html.erb:
> > > --------------
> > > <%
> > > record = list_record if list_record # compat with
> > > render :partial :collection
> > > dont_show_calculations ||= false
> > > if defined?(tr_class).nil?
> > > tr_class = cycle("", "even-record-")
> > > tr_class += "#{list_row_class(record)}" if
> > > respond_to? :list_row_class
> > > end
> > > url_options = params_for(:action => :list, :id => record.id)
> > > -%>
> > >
> > > <tr class="record <%= tr_class %>" id="<%= element_row_id(:action
> > > => :list, :id => record.id) %>">
> > > <% active_scaffold_config.list.columns.each do |column| %>
> > > <% column_value = get_column_value(record, column) -%>
> > >
> > > <td class="<%= column_class(column, column_value) %>" >
> > > <%= render_list_column(column_value, column, record) %>
> > > </td>
> > > <% end -%>
> > > <% if active_scaffold_config.action_links.any? {|link| link.type
> > > == :record } -%>
> > > <td class="actions">
> > > <%= render :partial => 'list_actions', :locals => {:record =>
> > > record, :url_options => url_options} %>
> > > </td>
> > > <% end -%>
> > > </tr>
> > >
> > > <% target_id = element_row_id(:action => :list, :id => record.id) -%>
> > > <script type="text/javascript">
> > > actions = new ActiveScaffold.Actions.Record(
> > > $$('#<%= target_id -%> a.action'),
> > > $('<%= target_id -%>'),
> > > $('<%= loading_indicator_id(:action => :record, :id => record.id) -
> > > %>'),
> > > {refresh_url: '<%= url_for params_for(:action => :row, :id =>
> > > record.id, :_method => :get, :tr_class => tr_class, :escape => false) -
> > > %>'}
> > > );
> > > <%= update_page do |page|
> > > page.replace active_scaffold_calculations_id, :partial =>
> > > 'list_calculations'
> > > end if not dont_show_calculations and
> > > active_scaffold_config.list.columns.any? {|c| c.calculation?} %>
> > > </script>
> > > --------------
> > >
> > > If possible, I'd ask if any AS developers could merge these changes
> > > into the master branch, since it requires a change to the core AS
> > > code.
> > >
> > > Thanks,
> > > -- Darien
> > >
> > > On Apr 30, 6:27 pm, dkindlund <[email protected]> wrote:
> > > > Hrm, so when putting this fix in place, it seems that when you
> > > > perform a nested drill down of any element, when you "close" that
> > > > nested table, the parent row somehow gets their "even-record" status
> > > > removed... is this a bug??
> > > >
> > > > -- Darien
> >
> > --
> > Sergio Cambra .:: entreCables S.L. ::.
> > Nicolás Guillén 6, locales 2 y 3. 50.018 Zaragoza
> > T) 902 021 404 F) 976 52 98 07 E) [email protected]
>
>
--
Sergio Cambra .:: entreCables S.L. ::.
Nicolás Guillén 6, locales 2 y 3. 50.018 Zaragoza
T) 902 021 404 F) 976 52 98 07 E) [email protected]
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"ActiveScaffold : Ruby on Rails plugin" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/activescaffold?hl=en
-~----------~----~----~----~------~----~------~--~---
