Closed Bug 1582375 Opened 5 years ago Closed 5 years ago

Urlbar should be focused when clicking its search icon

Categories

(Firefox :: Address Bar, defect, P2)

defect
Points:
2

Tracking

()

VERIFIED FIXED
Firefox 71
Iteration:
71.2 - Sept 16 - 29
Tracking Status
firefox71 --- verified

People

(Reporter: dao, Assigned: dao)

References

Details

Attachments

(1 file)

No description provided.

the urlbar is focused when clicking the search icon. but initiating input and opening the urlbar results popup is not triggered by focusing the urlbar, it's triggered by focusing the urlbar input container, which is to the right of the search icon, which is really an identity icon. to change that you'd have to deal with some inconsistencies. the identity icon, tracking protection icon, and other icons (notifications, permissions, etc.) have their own click events already. the search icon is the only exception but it's not its own icon, it is just the identity icon with pointer-events: none. one way to make this happen would be to give the search icon its own unique object and make it a child of urlbar-input-container. then urlbar input container would cover the whole left side of the urlbar when [pageproxystate="invalid"], and that region would be available to register click events.

but this would be sort of inconsistent behavior, don't you think? it might be confusing to users that sometimes, they can click the icons to the left of the url to start typing a search/url, but other times, clicking those icons will only open their corresponding popups. also, the behavior of the icons would then be changing all the time. when you first load a page, [pageproxystate="valid"] and you see a lock icon and a tracking icon. but when you type into the urlbar, it becomes invalid and those icons disappear and would then be replaced by the search icon, which you could click to open the urlbar. so you might click the icon and type a url, and hit enter. and then when the page loads you might click the icon again to repeat the process, but now you can't, because pageproxystate has returned to valid and the search icon has been replaced with the identity icon.

it's a bit of a compromise but imo those icon popups are more valuable than the click behavior of the search icon. especially now with identity labels being hidden, the way for average users to get detailed information about the security of a page is to click the identity icon and the tracking protection icon. that will give them information the icons cannot convey pictorially, like permissions, the certificate, which elements are insecure, which trackers or malicious scripts have been detected, etc. there might be a better compromise that i haven't thought of though.

See Also: → 1583535
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Depends on: 1584167
Iteration: --- → 71.2 - Sept 16 - 29
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/65f2f41ae481
Focus the urlbar when clicking directly on the input container (this includes the search glass which doesn't consume pointer events). r=harry
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71
Blocks: 1584507
Depends on: 1579004
No longer depends on: 1579004
Blocks: 1584938

I have reproduced this bug with Nightly 71.0a1 (2019-09-19) on windows 10, 64 bit!

The fix is now verified on Latest Nightly 71.0a1

Build ID : 20190930095343
User Agent : Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:71.0) Gecko/20100101 Firefox/71.0

QA Whiteboard: [bugday-20190925]
Flags: needinfo?(cristian.comorasu)
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Flags: needinfo?(cristian.comorasu)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: