Frédéric Buclin Reporter
	Description • 18 years ago

Mozilla/5.0 (X11; U; Linux i686; fr; rv:1.8.1b1) Gecko/20060815 BonEcho/2.0b1

Look at http://landfill.bugzilla.org/qa222my5/attachment.cgi?id=26 and http://landfill.bugzilla.org/qa222my5/attachment.cgi?id=25 for two screenshots of the pref panel. On the first one, the available height seems to be fixed and ignores wrapped lines. Probably it should adapt itself or at least be taller.

	Cédric Corazza
	Comment 1 • 18 years ago

I had a chat on IRC with LpSolit and I couldn't reproduce the bug with the same system police parameters on my config. I encountered that kind of problem before and removing *.mfasl files solved it. Apparently not in that case. 

	Mike Beltzner [:beltzner, not reading bugmail]
	Comment 2 • 18 years ago

Plussing, but can we please get people trying to reproduce this? We can't ship beta 2 with a broken prefpanel ...

	Jeff Walden [:Waldo] Assignee
	Comment 3 • 18 years ago

I think this is an l10n problem.  The window size (height and width) are fully controlled by localizable preferences, and if languages require greater width to accommodate lengthy strings, they should use a greater width via the appropriate entities.

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/browser/components/preferences/preferences.xul&rev=1.10&root=/cvsroot&mark=77-85#73

	Jeff Walden [:Waldo] Assignee
	Comment 4 • 18 years ago

By the way, for the first screenshot I suspect you're seeing the underlying bug which triggered bug 345572 (wrapping checkboxes screw up containing box height, at a guess), but I haven't tested to be sure.

	Frédéric Buclin Reporter
	Comment 5 • 18 years ago

Talking about the height, making the pref panel taller doesn't help, at least in the first URL from comment 0. The vertical space available for text doesn't adapt to match wrapped lines. Why keeping it so small?

	Jeff Walden [:Waldo] Assignee
	Comment 6 • 18 years ago

It's really either this or an entity to control height for every single checkbox in preferences, and I (and localizers, I'm sure) would rather not do that.

	Jeff Walden [:Waldo] Assignee
	Comment 8 • 18 years ago

Apparently I still don't know enough about XUL.

	Mike Connor [:mconnor]
	Comment 9 • 18 years ago

Comment on attachment 234506 [details] [diff] [review]
Better hack

That'll work, ship it!

	Jeff Walden [:Waldo] Assignee
	Comment 10 • 18 years ago

Checked in branch/trunk.  The width issue should be something localizers should handle as needed.

	Frédéric Buclin Reporter
	Comment 11 • 18 years ago

The UI is fine with:
Mozilla/5.0 (X11; U; Linux i686; fr; rv:1.8.1b2) Gecko/20060821 BonEcho/2.0b2

I know this fix is responsible for bug 349343, but this particular issue is fixed. Marking VERIFIED.