Code review comment for lp:~zsombi/ubuntu-ui-toolkit/combobutton

Revision history for this message
Zsombor Egri (zsombi) wrote :

> I think the blue contents for the combobutton in the gallery is quite ugly:
> http://ubuntuone.com/4jy8WPRYPLbLTez89wzqRh
> also it does not really show how it should be used, with listitems that have
> alternative options to click on. Please change the example in the gallery.

Agreed. I'll add a Column with items into it.
>
>
> I also don't really like the colors used in the button itself, but I guess we
> need design input for that.

The colors I got from design. We were supposed to get UI facelift again for all the components. If they don't like the design, they can change it. Actually I was thinking to expose these colors/font properties so user can customize them freely.

>
> Finally, the ^/V just "flips" when you click it, a rotation animation such as
> the ValueSelector has looks more sleek.

That's the point. The chevron should not be animated while the dropdown toggles. The maximum we could do is to fade the image transition, however atm we don't have separate up/down chevrons. But we can make one, so I'll update the UI.

« Back to merge proposal