1393054 - Awesome bar hover state (border, box-shadow) should be consistent with back button

Status: VERIFIED FIXED

(Firefox :: General, enhancement, P1)

Description

[Eric Pang [:epang] UX]

Currently the hover state state of the awesome bar is slightly darker then the back button.

Can we update and align the stroke and drop shadow to the back button specifications?

Comment 1

[Dale Harvey (:daleharvey)]

Attachment: Bug 1393054 - Remove extra box-shadow from urlbar.

Review commit: https://reviewboard.mozilla.org/r/175408/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/175408/

Comment 2

[Dale Harvey (:daleharvey)]

I specifically set that box shadow in https://bugzilla.mozilla.org/show_bug.cgi?id=1352366 however I cant remember why, its not in the spec and not mentioned in the comments as far as I can see. 

Thats the only difference between the url bar and the back button, they otherwise match the spec

Comment 3

[Dão Gottwald [:dao]]

Comment on attachment 8903686 [details]
Bug 1393054 - Remove extra box-shadow from urlbar.

https://reviewboard.mozilla.org/r/175408/#review180702

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7c4c81a8b699
Remove extra box-shadow from urlbar. r=dao

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/7c4c81a8b699

Comment 6

[Valentina Claudia Ona]

I verify this issue on Windows 10 x64 using Nightly 58.0a1 with build ID 20171001220301 and the conclusion was that the hover state of the awesome bar, the color is darker (#F0F0F0) than the build with the old version of 2017.08.24 (#F2F2F2).
This is the result you are expecting?

Comment 7

[Valentina Claudia Ona]

Attachment: aa.bmp

I tested this problem using Nightly 58.0a1 with the build ID 20171002100134 on Windows 10 x64 and Windows 7 x64 and I see no difference, but on Mac OS 10.12 and Ubuntu 16.04, the problem is resolved.

Comment 8

[Valentina Claudia Ona]

Sorry, about my mistake! Please ignore this comment.

Comment 9

[Dão Gottwald [:dao]]

(In reply to Valentina Claudia Ona from comment #6)
> I verify this issue on Windows 10 x64 using Nightly 58.0a1 with build ID
> 20171001220301 and the conclusion was that the hover state of the awesome
> bar, the color is darker (#F0F0F0) than the build with the old version of
> 2017.08.24 (#F2F2F2).
> This is the result you are expecting?

No, the location bar's hover box-shadow was darker, should be lighter now to match the back button.

Comment 10

[ovidiu boca[:Ovidiu]]

I tested on Mac OS X 10.10 with a build before the fix and with the latest Nightly build 58.0a1(2017-10-03) and I don't see any differences. Eric, can you please give us more details about this issue, on what OS did you encounter this issue and other details that can help us verify this. Thanks

Comment 11

[Eric Pang [:epang] UX]

Hi Ovidiu, it was a subtle change but it looks good now.  Thanks!

Comment 12

[ovidiu boca[:Ovidiu]]

Thanks Eric for your reply, can you please tell me on what FF version did you test this, I assume FF Nightly 58.0a1. But just to be on the same page we would appreciate if you can make a screen recorder or a print screen to see how to verify this issue. Thanks

Comment 13

[Dão Gottwald [:dao]]

Since Eric reported the bug, I think his word is good enough to mark this verified. Thanks!