Closed Bug 1293308 Opened 3 years ago Closed 3 years ago

System font setting respected by location bar but ignored by location bar's autocomplete drop-down list

Categories

(Firefox :: Address Bar, defect, P1)

48 Branch
x86_64
Linux
defect

Tracking

()

VERIFIED FIXED
Firefox 51
Tracking Status
firefox49 --- verified
firefox50 --- verified
firefox51 --- verified

People

(Reporter: icuin2me, Assigned: adw)

References

(Depends on 1 open bug)

Details

(Keywords: regression, Whiteboard: [fxsearch])

Attachments

(1 file)

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.
OS: Unspecified → Linux
Hardware: Unspecified → x86_64
Component: Untriaged → Location Bar
Summary: System font setting respected by urlbar but ignored by urlbar's autocomplete drop-down list → System font setting respected by location bar but ignored by location bar's autocomplete drop-down list
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.
Priority: -- → P1
Whiteboard: [fxsearch]
Blocks: 1181078
Duplicate of this bug: 1292920
I haven't tested this yet myself, but confirming since there was a dupe.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee: nobody → adw
Status: NEW → ASSIGNED
Keywords: regression
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.
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?
Attachment #8784657 - Flags: review?(mak77) → review+
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.
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.
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
https://hg.mozilla.org/mozilla-central/rev/f32e2e8eb298
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
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
Attachment #8784657 - Flags: approval-mozilla-beta?
Attachment #8784657 - Flags: approval-mozilla-aurora?
Hello Ajay, could you please verify this issue is fixed as expected on a latest Nightly build? Thanks!
Flags: needinfo?(icuin2me)
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+
Attachment #8784657 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
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.
Attachment #8784657 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Depends on: 1299777
Flags: qe-verify+
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.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Flags: needinfo?(icuin2me)
You need to log in before you can comment on or make changes to this bug.