Code review comment for lp:~rharding/juju-gui/new-controls

Revision history for this message
Curtis Hovey (sinzui) wrote :

Thank you Rick. These changes LGTM with the additional changes that I
can see are already added at Lp.

I see the discussion about widget vs view. I think widget is fine at
this time. This is clearly more than a view. The widget might be too
heavy for this use, but that may change in the next few weeks. That is
to say, We are making this late change to accommodate a late design.
Given the urgency to get the UI looking good, I think it is wiser to
review what we don't need at the end of July.

https://codereview.appspot.com/10877043/

« Back to merge proposal