[Ovirt-devel] [PATCH] Updates to treenav

[Scott Seago]

Jason Guiditta wrote:
> *  Got the 'You are here' bit working in the tree.  As always with both
> ruby and jquery, this was much simpler than it seemed.  The biggest
> thing we were missing was that we were not passing back an 'expanded'
> value in the json, which the async.js looks for.  That plus a little
> work to determine properly what params to pass in were the main things.
> *  Merged in some new css from Tim, so it should look much closer to the
> mockups, though I am not sure what we are doing for the links css-wise,
> but that is easy enough to fix later.
> *  Still trying to figure out where to put the hook so the animation
> doesn't happen until the data is received back from the server.  I have
> a pretty good idea, just haven't quite pulled it together yet.  In the
> interim, I cleaned it up so it doesn't show the placeholder, and instead
> just have it opening w/o the animation.  I'll add it back in as soon as
> I figure out the correct way to do it.
>   
Looks fine to me
ACK

Scott