1187846 - Badged button's minimum width is wrong

Status: RESOLVED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[Matteo Ferretti [:zer0] [:matteo]]

Attachment: small badge

This is a regression probably caused by Bug 1029937.
Previously, we had a minimum width set for the badges, where now it seems we doesn't, causing a very narrow and small badges if we set single digit content. See the attachment, compared to: https://mdn.mozillademos.org/files/9803/badge.png

Comment 1

[Matteo Ferretti [:zer0] [:matteo]]

Attachment: badges comparison

Added a better screenshot for comparison. On the left of the screenshot, we have the UX mockup and how it was used to be. On the right, there is the current badge after the fix of bug 1029937.
After a couple of check, it seems to me that `min-width` is totally ignored on the XUL label element that is used for the badge, as well as `max-width`. Not sure if it's depends by the stack or what else.

Neil, you've implemented the stack solution, do you have any idea about that?

If we didn't find a good solution for that, to mitigate the difference we could just add some extra padding on right and left, like 3px or 4px instead of 2px.

Brian, what do you think?

Comment 2

[[e-mail]]

Attachment: Possible patch

Comment 3

[[e-mail]]

Bugzilla, please let me finish filling the attachment form before submitting it. Thanks!

Comment 4

[[e-mail]]

Attachment: Possible patch

Stack layout doesn't seem to honour min/max sizes for unstretched elements.

Comment 5

[[e-mail]]

(I can work around the problem by wrapping the label in a box; the label's minimum width then becomes the box's preferred width.)

Comment 6

[Brian Grinstead [:bgrins]]

It'd be great if we could fix this in the stack layout.  Alternatively adding a few extra px padding on the side to prevent the really narrow badge seems reasonable.

Comment 7

[Matteo Ferretti [:zer0] [:matteo]]

Thanks for the quick response guys! I'm assigning this patch to you, Neil.

Comment 8

[Neil Deakin]

Comment on attachment 8639925 [details] [diff] [review]
Possible patch

Could use a test though.

Comment 9

[[e-mail]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/71d335f1a4a3

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/71d335f1a4a3

Comment 11

[Matteo Ferretti [:zer0] [:matteo]]

Notice that the Look & Feel is still wrong, compared to the screenshot, but now that the `min-width` is working, it would be fixed by bug 1188001.