Closed Bug 1454264 Opened 6 years ago Closed 6 years ago

New Tab Page Search Bar Focus effect looks wrong on Mac OS X

Categories

(Firefox :: New Tab Page, defect)

All
macOS
defect
Not set
normal

Tracking

()

RESOLVED INVALID
Tracking Status
firefox-esr52 --- unaffected
firefox59 --- unaffected
firefox60 --- unaffected
firefox61 - wontfix

People

(Reporter: mehmet.sahin, Unassigned)

References

Details

(Keywords: regression)

Attachments

(2 files)

Attached image Actual vs Expected
Nightly 61.0a1 (2018-04-15) (64-Bit)
macOS 10.12.6

(1) Open a NTP
(2) Focus the Search Bar.

Actual: Wrong Focus Ring color.

Expected: Correct color.

A screenshot is attached.

This is a regression in recent Nightly.
10:43.49 INFO: Last good revision: 1344ca26ef0aec57cb0177f7089287b263c7a52b
10:43.49 INFO: First bad revision: 9896401414cb3c7cb5f1118b307b0267212c8aad
10:43.49 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=1344ca26ef0aec57cb0177f7089287b263c7a52b&tochange=9896401414cb3c7cb5f1118b307b0267212c8aad

Looks like bug 1454074 caused this.

The Top Site edit fields also have the same hover effect. Should those be changed as well?
Blocks: 1454074
Keywords: regression
OS: Unspecified → Mac OS X
Hardware: Unspecified → All
Summary: Regression: NTP Search Bar Focus Ring looks wrong → New Tab Page Search Bar Focus effect looks wrong on Mac OS X
Hi, 

The new focus style is intentional (double outline) and also intentional in the Top Site edit field. This is to reflect the DSG here: https://design.firefox.com/photon/components/input-fields.html
(In reply to Amy Lee [:amylee] UX from comment #2)
> Hi, 
> 
> The new focus style is intentional (double outline) and also intentional in
> the Top Site edit field. This is to reflect the DSG here:
> https://design.firefox.com/photon/components/input-fields.html

Thanks for your feedback. If this is intentional, maybe the Dropdown (especially the top grey border) can be placed under the blue focus ring? Please see the enclosed screenshot.

Thanks :-)
(In reply to Mehmet from comment #3)
> Created attachment 8968295 [details]
> Bildschirmfoto 2018-04-16 um 21.09.51.png
> 
> (In reply to Amy Lee [:amylee] UX from comment #2)
> > Hi, 
> > 
> > The new focus style is intentional (double outline) and also intentional in
> > the Top Site edit field. This is to reflect the DSG here:
> > https://design.firefox.com/photon/components/input-fields.html
> 
> Thanks for your feedback. If this is intentional, maybe the Dropdown
> (especially the top grey border) can be placed under the blue focus ring?
> Please see the enclosed screenshot.
> 
> Thanks :-)

Hi, 

Yes this will be fixed in the next update. There's some visual changes I'm currently working on for the search bar in New Tab.
This change is by design as per comment 2. amylee, if there's a bug for the dropdown position, please link it.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
The adjusting dropdown position is fixed as part of bug 1454263.
See Also: → 1454263
(In reply to Ed Lee :Mardak from comment #6)
> The adjusting dropdown position is fixed as part of bug 1454263.

Thanks for the info :-)
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: