Ajay Anand Reporter
	Description • 8 years ago
str

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:48.0) Gecko/20100101 Firefox/48.0
Build ID: 20160728203720

Steps to reproduce:

STEP 1: Open terminal and issue the following command,
gsettings set org.gnome.desktop.interface font-name 'Ubuntu 10'
Observation 1: Urlbar font size is small.
Observation 2: Urlbar popup/drop-down font size is small.

STEP 2: Open terminal and issue the following command,
gsettings set org.gnome.desktop.interface font-name 'Ubuntu 20'
Observation 1: Urlbar font size is big.
Observation 2: Urlbar popup/drop-down font size is small. (THIS IS THE ISSUE)




Actual results:

When system font size was increased from 10 to 20, urlbar font size increased but urlbar autocomplete drop-down list font size did not increase.


Expected results:

When system font size was increased from 10 to 20, urlbar font size increased and urlbar drop-down list font size should also have increased just as it used to be in previous versions of Firefox.

	Drew Willcoxon :adw Assignee
	Comment 1 • 8 years ago

This might be because .ac-title and .ac-url have font-sizes defined in terms of px.  Before the redesign, .ac-comment and .ac-url-text had em font-sizes.

This may affect people with impaired vision, and the fix should be simple, and we should uplift it, so we should probably look into this soon.

	Drew Willcoxon :adw Assignee
	Comment 3 • 8 years ago

I haven't tested this yet myself, but confirming since there was a dupe.

	Drew Willcoxon :adw Assignee
	Comment 5 • 8 years ago

This goes back to using the em sizes we used before the redesign: https://hg.mozilla.org/mozilla-central/diff/061165ac1ff9/browser/themes/linux/browser.css

It retains the size difference between titles and URLs from the new design but also correctly resizes when doing the STR in comment 0.

	Marco Bonardo [:mak]
	Comment 6 • 8 years ago
mozreview-review

Comment on attachment 8784657 [details]
Bug 1293308 - System font setting respected by location bar but ignored by location bar's autocomplete drop-down list.

https://reviewboard.mozilla.org/r/74008/#review72404

This is fine for Linux.
What about Windows? We were using em on Windows too before the change, and while I think it's not possible anymore to change the font in 8, 8.1 and 10, it is still possible in Windows 7, that we support. What do you think?

	Drew Willcoxon :adw Assignee
	Comment 7 • 8 years ago

Ah OK, I tried changing the system font size on 10 and couldn't figure out how to do it.  I'll try on 7 then.

	Drew Willcoxon :adw Assignee
	Comment 8 • 8 years ago

Interestingly the current Nightly works on Windows 7 already, even with the px units.  There are three options for font size that I can find: in Control Panel > Appearance and Personalization > Display.  And the popup text is adjusted based on the size I pick.  So I'll land this as is.

	Pulsebot
	Comment 9 • 8 years ago

Pushed by dwillcoxon@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f32e2e8eb298
System font setting respected by location bar but ignored by location bar's autocomplete drop-down list. r=mak

	Ryan VanderMeulen [:RyanVM]
	Comment 10 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/f32e2e8eb298

	Drew Willcoxon :adw Assignee
	Comment 11 • 8 years ago

Comment on attachment 8784657 [details]
Bug 1293308 - System font setting respected by location bar but ignored by location bar's autocomplete drop-down list.

Approval Request Comment
[Feature/regressing bug #]: New urlbar popup design, bug 1181078
[User impact if declined]: Linux users who have non-default system font sizes will see incorrect font size in urlbar popup (e.g., users with accessibility needs)
[Describe test coverage new/current, TreeHerder]: Manual testing
[Risks and why]: Low risk, a small CSS-only Linux-only change
[String/UUID change made/needed]: None

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 12 • 8 years ago

Hello Ajay, could you please verify this issue is fixed as expected on a latest Nightly build? Thanks!

	Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)
	Comment 13 • 8 years ago

Comment on attachment 8784657 [details]
Bug 1293308 - System font setting respected by location bar but ignored by location bar's autocomplete drop-down list.

CSS only, low-risk, Aurora50+

	Ryan VanderMeulen [:RyanVM]
	Comment 14 • 8 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-aurora/rev/ca2340129749

	Gerry Chang [:gchang]
	Comment 15 • 8 years ago

Comment on attachment 8784657 [details]
Bug 1293308 - System font setting respected by location bar but ignored by location bar's autocomplete drop-down list.

This patch fixes a UI issue of autocomplete drop-down list in location bar. Take it in 49 beta.

	Ryan VanderMeulen [:RyanVM]
	Comment 16 • 8 years ago
bugherder uplift

https://hg.mozilla.org/releases/mozilla-beta/rev/58ec38e8bfd6

	Petruta Horea [:phorea], Desktop QA
	Comment 17 • 8 years ago

Verified as fixed with Firefox 49 beta 9, latest Aurora 50.0a2 2016-09-02 and latest Nightly 51.0a2 2016-09-01 under Ubuntu 16.04 64-bit.