Closed
Bug 1211217
Opened 9 years ago
Closed 9 years ago
Show the URL bar dropmarker on toolbar hover instead of location bar hover
Categories
(Firefox :: Theme, defect)
Firefox
Theme
Tracking
()
RESOLVED
FIXED
Firefox 44
People
(Reporter: jaws, Assigned: jaws)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
jaws
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Updated•9 years ago
|
status-firefox43:
--- → affected
Assignee | ||
Comment 1•9 years ago
|
||
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•9 years ago
|
||
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.
Attachment #8669385 -
Flags: review?(dao)
Assignee | ||
Comment 3•9 years ago
|
||
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.
Attachment #8669385 -
Attachment is obsolete: true
Attachment #8670540 -
Flags: review?(dao)
Comment 4•9 years ago
|
||
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
Attachment #8670540 -
Flags: review?(dao) → review+
Assignee | ||
Comment 5•9 years ago
|
||
Thanks, carrying forward r+
Attachment #8670540 -
Attachment is obsolete: true
Attachment #8670890 -
Flags: review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
You need to log in
before you can comment on or make changes to this bug.
Description
•