1211217 - Show the URL bar dropmarker on toolbar hover instead of location bar hover

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Jared Wein [:jaws] (please needinfo? me)]

Bug 1203494 made the URL bar dropmarker only appear when the location bar is hovered, but that requires high precision and some specific directional movement of the mouse to trigger it with high possibility of stopping the mouse where the dropmarker will appear.

Also, as a user who frequently uses the dropmarker, I find that it disappears too quickly if I accidentally overshoot the location bar and the mouse stops on the tabstrip above the navigation toolbar.

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch

This patch is an improvement over what we currently are doing, but the #navigator-toolbox doesn't get :hover when hovering over the #TabsToolbar in an area that is absent of tabs (at least on Windows). I don't know what we can do about that, but if we can't do anything about that at least this patch makes it incrementally better.

Comment 2

[Dão Gottwald [:dao]]

Comment on attachment 8669385 [details] [diff] [review]
Patch

> .urlbar-history-dropmarker {
>   -moz-appearance: toolbarbutton-dropdown;
>   transition: opacity 0.15s ease;
>+  opacity: 0;
> }
> 
>-#urlbar:not(:hover) > .urlbar-textbox-container > .urlbar-history-dropmarker {
>-  opacity: 0;
>+#navigator-toolbox:hover .urlbar-textbox-container > .urlbar-history-dropmarker {
>+  opacity: 1;
> }

I don't understand why you changed the logic here rather than just updating the selector.

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch v2

Sorry, I had initially approached the patch by writing selectors for each toolbar and inverted the logic until I realized I wouldn't be able to write a selector for the #PersonalToolbar which is after the #navbar. When I switched it to using the #navigator-toolbox I didn't think about reverting the logic to the pre-patch approach.

Comment 4

[Dão Gottwald [:dao]]

Comment on attachment 8670540 [details] [diff] [review]
Patch v2

>-#urlbar:not(:hover) > .urlbar-textbox-container > .urlbar-history-dropmarker {
>+#navigator-toolbox:not(:hover) .urlbar-textbox-container > .urlbar-history-dropmarker {
>   opacity: 0;
> }

".urlbar-textbox-container > " is redundant now that you're not using the child selector all the way.

r=me with this removed

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch for check-in

Thanks, carrying forward r+

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/78cf8ddd23cb

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/78cf8ddd23cb