Closed Bug 1468671 Opened Last year Closed Last year
Tracking Protection in identity popup is broken for longer localizations
This is how it currently looks in Italian: the entire information about tracking protection being ON or OFF is missing, and OFF is DISATTIVA in Italian…
P.S. Ignore the typo if you can read Italian, it's already been fixed
One more issue (label for the button is overflowing).
Thanks for catching this. I hope just making things wrap will fix it
So, making the text wrap unfortunately doesn't really seem to work with the flex model, I can't get the container to grow with its children as I want it to. Hence, I switched to using CSS Grid instead.
Comment on attachment 8985875 [details] Bug 1468671 - Enable identity popup headlines to span multiple lines. Maybe I'm missing something, but can't you just use the box model and set flex="1" on the labels and maybe align="center" on the container? Seems to work fine for me, although you may want to double check the desired element alignment. I tested this patch and it is broken when the permissions header wraps.
Attachment #8985875 - Flags: review?(nhnt11)
Let me try that, thanks!
So I'm 95% sure I tried something like this earlier, probably missed some detail. It works now. Thank you for the suggestion, Paolo!
Comment on attachment 8985875 [details] Bug 1468671 - Enable identity popup headlines to span multiple lines. https://reviewboard.mozilla.org/r/251376/#review258596 ::: browser/components/controlcenter/content/panel.inc.xul:60 (Diff revision 2) > <!-- Tracking Protection Section --> > <hbox id="tracking-protection-container" > class="identity-popup-section" > when-connection="not-secure secure secure-ev secure-cert-user-overridden file"> > <vbox id="tracking-protection-content" flex="1"> > <hbox class="identity-popup-preferences-button-container"> Remove the two instances of the unused "identity-popup-preferences-button-container" class.
Attachment #8985875 - Flags: review?(paolo.mozmail) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/6fd500bb5dd8 Enable identity popup headlines to span multiple lines. r=Paolo
QA Whiteboard: [good first verify]
You need to log in before you can comment on or make changes to this bug.