The default bug view has changed. See this FAQ.

Truncated "Learn more" and button in geolocation doorhanger (does not fit, too long, Russian, Esperanto)

VERIFIED FIXED in Firefox 7

Status

()

Firefox
Theme
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: Aleksej, Assigned: Margaret)

Tracking

(Depends on: 1 bug)

Trunk
Firefox 7
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [bugday-2011-05-27], URL)

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 535688 [details]
screenshot of the bug in Esperanto and Russian Aurora 20110527 for Linux-x86_64 (with a Shiki-Colors GTK theme)

It seems that the button in the geolocation doorhanger is right-aligned at the center of the doorhanger, so, when it is long, it pushes the "Learn more" text out to the left (e.g. in Esperanto), and can itself be truncated (e.g. in Russian).

Nightly
WFM: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:2.0b8pre) Gecko/20101203 Firefox/4.0b8pre
stretched icon: Mozilla/5.0 (X11; Linux x86_64; rv:2.0b8pre) Gecko/20101205 Firefox/4.0b8pre
stretched icon: Mozilla/5.0 (X11; Linux x86_64; rv:2.0b8pre) Gecko/20101206 Firefox/4.0b8pre
bug: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:2.0b8pre) Gecko/20101208 Firefox/4.0b8pre

Beta:
bug: Mozilla/5.0 (X11; Linux x86_64; rv:5.0) Gecko/20100101 Firefox/5.0

Aurora:
bug: Mozilla/5.0 (X11; Linux x86_64; rv:2.2a1pre) Gecko/20110412 Firefox/4.2a1pre
bug: Mozilla/5.0 (X11; Linux x86_64; rv:5.0a2) Gecko/20110506 Firefox/5.0a2
(Reporter)

Updated

6 years ago
Summary: Truncated "Learn more" and button in geolocation doorhanger → Truncated "Learn more" and button in geolocation doorhanger (does not fit, too long, Russian, Esperanto)
Bug 615481 having used a px value for the description's max-width is probably at fault here.
Assignee: nobody → margaret.leibovic
Blocks: 615481
Component: General → Theme
QA Contact: general → theme
(Assignee)

Comment 2

6 years ago
Created attachment 536176 [details] [diff] [review]
patch

I agree that using em instead of px for the max-width will make the notification wider to accommodate more text, but it looks like something else is going wrong in this screenshot. It looks like the vbox that contains the close button is taking up more space than it should. Even if the description text wraps, the menubutton should still be aligned farther to the right. I guess we can investigate that further if this fix doesn't fix the problem.

I somewhat arbitrarily picked 24em because that looks right on my platform. Feel free to offer suggestions if there's a smarter way to choose this value.
Attachment #536176 - Flags: review?(dao)

Updated

6 years ago
Attachment #536176 - Flags: review?(dao) → review+
(Assignee)

Comment 3

6 years ago
http://hg.mozilla.org/mozilla-central/rev/de924c37271e
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 7

Comment 4

6 years ago
Erm, I only noticed this because of seeing the checkin - but em is really a (line) height unit, widths should use ch units that are tied to character widths.
Yeah, I hear people say that a lot, but no one has ever explained to me why that actually matters in practice.

Comment 6

6 years ago
(In reply to comment #5)
> Yeah, I hear people say that a lot, but no one has ever explained to me why
> that actually matters in practice.

We had a number of cases where dialogs would actually become too narrow if people had a system font that has wider than high characters. Those are edge cases, but using ch for widths we can deal with that nicely. Also, we saw that due to different OSes using different system fonts (with different character widths), what looks like a reasonable width on one could look badly off on the other. The "em" unit mostly corresponds to the "font size", which actually is the default line height in the end, and depending on the font, a width of 30em might fit 60 or 20 characters...

Comment 7

6 years ago
Verified issue on both locals Esperanto and Russian.

Mozilla/5.0 (X11; Linux x86_64; rv:6.0a2) Gecko/20110601 Firefox/6.0a2 -eo

Mozilla/5.0 (X11; Linux x86_64; rv:6.0a2) Gecko/20110601 Firefox/6.0a2 -ru

The Geolocation doorhangers are now complete.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.