Closed Bug 1273943 Opened 8 years ago Closed 8 years ago

Emphasize search terms in awesomebar results using bold face

Categories

(Firefox :: Address Bar, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 49
Tracking Status
firefox48 --- verified
firefox49 --- verified

People

(Reporter: adw, Assigned: adw)

References

Details

Attachments

(1 file)

http://cl.ly/3x3v1518181v

We want to land this in Nightly and uplift it to 48.
font-weight: 400; normal
font-weight: 500;
font-weight: 600;
font-weight: 700; bold

What are we supposed to use? I think stephen said "lighter bold", that maybe maps to 600?
I used 500. Which works on platforms that have that weight and seems to bump up to bold if they don't.
(In reply to Stephen Horlander [:shorlander] from comment #2)
> I used 500. Which works on platforms that have that weight and seems to bump
> up to bold if they don't.

mdn doesn't seem to agree

https://developer.mozilla.org/en-US/docs/Web/CSS/font-weight

If a weight of exactly 500 is given, then 400 is used. If 400 is not available, then the heuristic for font weights greater than 500 is used.
This means that for fonts that provide only normal and bold, 100-500 are normal, and 600-900 are bold.
Oh, you are right. I did use 600 :)
Blocks: 1262514, 1262606
Blocks: 1262526
Comment on attachment 8754974 [details]
MozReview Request: Bug 1273943 - Emphasize search terms in awesomebar results using bold face. r?mak

https://reviewboard.mozilla.org/r/54314/#review51194
Attachment #8754974 - Flags: review?(mak77) → review+
https://hg.mozilla.org/mozilla-central/rev/10a29aed7ab0
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Comment on attachment 8754974 [details]
MozReview Request: Bug 1273943 - Emphasize search terms in awesomebar results using bold face. r?mak

Approval Request Comment
[Feature/regressing bug #]: awesomebar redesign
[User impact if declined]: no polished ui
[Describe test coverage new/current, TreeHerder]: Nightly
[Risks and why]: very low risk, simple css changes
[String/UUID change made/needed]: none
Attachment #8754974 - Flags: approval-mozilla-aurora?
Comment on attachment 8754974 [details]
MozReview Request: Bug 1273943 - Emphasize search terms in awesomebar results using bold face. r?mak

Polish with small risk, taking it!
Attachment #8754974 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
I have reproduced this bug on Nightly 49.0a1 (2016-05-18) on Ubuntu 16.04 LTS, 64 bit!

The bug's fix is now verified on Latest Aurora 48.0a2!

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

Also the bug’s fix is now verified on Latest Nightly 49.0a1!

Build ID: 20160531030258
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:49.0) Gecko/20100101 Firefox/49.0
QA Whiteboard: [bugday-20160601]
I have reproduced this bug with Firefox Nightly 49.0a1 (Build ID: 20160518030234) on 
Windows 8.1, 64-bit.

Verified as fixed with Latest Firefox Release 48.0 (Build ID: 20160726073904)
Mozilla/5.0 (Windows NT 6.3; WOW64; rv:48.0) Gecko/20100101 Firefox/48.0

Verified as fixed with Latest Firefox beta 49.0b5 (Build ID: 20160818050015)
Mozilla/5.0 (Windows NT 6.3; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0

As this bug is also verified on Ubuntu (comment 12), I am marking this as verified!
Status: RESOLVED → VERIFIED
QA Whiteboard: [bugday-20160601] → [bugday-20160601][bugday-20160817]
You need to log in before you can comment on or make changes to this bug.