Okay, thanks for the input.
On Jan 19, 1:13 pm, "Bill Walton" <bill.wal...@charter.net> wrote:
> Hi David,
>
> David wrote:
>
> <snip>
>
> > My question is if this is the best way of doing it?
>
> Depends on your definition of 'best'. It works and it's easy to follow the
> code. Good enough for me.
>
> > Im wondering if there is some way to use hide/show html or something,
>
> Sure, but you'd have to rework your view so you'd have a separate set of
> <div>s for the originals which you could hide/show, and a second set into
> which you'd render the edit partials.
>
> > b/c the second database query seems redundant, seeing as how the
> > appointment html existed in the first place. Thanks for any advice.
>
> DRY is not an absolute. Neither is readability. Both are components of
> maintainability. Go for balance. IMHO, if there's a tie, readability wins.
> YMMV.
>
> HTH,
> Bill
--~--~---------~--~----~------------~-------~--~----~
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-talk@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
-~----------~----~----~----~------~----~------~--~---