Josh _ wrote: > As an unrelated, but hopefully helpful aside, I'd also recommend not > duplicating your markup in both the "if" and "else" sections. Doing > things > this way quickly gets hard to read and maintain. Better might be: > > <% if (article.gender == "Male") then color = '#d1e1fa' else color = > '#111111' end %> > <table style="background-color: <%= color %>">
Better yet: <table class="<%= article.gender.downcase %>"> ...then define the colors in your CSS. The style attribute leads to unmaintainable HTML and should be avoided like the plague. Best, -- Marnen Laibow-Koser http://www.marnen.org mar...@marnen.org -- Posted via http://www.ruby-forum.com/. -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Talk" group. To post to this group, send email to rubyonrails-t...@googlegroups.com. To unsubscribe from this group, send email to rubyonrails-talk+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/rubyonrails-talk?hl=en.