On 20 February 2015 at 00:01, TTambe <tambe...@gmail.com> wrote: > I actually ended up doing this which worked: > > <% last_artist ||= nil %> > > <table id="artist"> > <tr> > <th>Artist</th> > <th></th> > <th>Album</th> > </tr> > <% @pin_albums.each do |pin| %> > <% if last_artist != pin.artist %> > <tr> > <td><%= pin.artist %></td> > <td><%= link_to image_tag(pin.image), pin %></td> > <td><%= link_to pin.album, copy_pin_path(pin) %></td> > </tr> > <% else %> > <tr> > <td></td> > <td><%= link_to image_tag(pin.image), pin %></td> > <td><%= link_to pin.album, copy_pin_path(pin) %></td> > </tr> > <% end %> > <% last_artist = pin.artist %> > <% end %>
You are right in that the key was to initialise last_artist at an appropriate scope. There are an awful lot of repeated lines there, Suppose you wanted to change the detail of the contents of the last cell in each row, you would have to change it in two places. You should put the if statement round just the first cell not round the whole row. so for that cell use something like <td> <%= (last_artist == pin.artist) ? '' : pin.artist %> </td> Or use an if statement inside the <td> </td> if you prefer. That is two single quotes in the line above of course. Colin -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-talk+unsubscr...@googlegroups.com. To post to this group, send email to rubyonrails-talk@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/rubyonrails-talk/CAL%3D0gLuBuJUBego%3Dq_5jDGaBRCA0rVHRwHdZDwjKrg665Hbqiw%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.