Top sites PIN option is always highlighted
Categories
(Firefox :: New Tab Page, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox-esr68 | --- | unaffected |
firefox67 | --- | unaffected |
firefox68 | --- | unaffected |
firefox69 | --- | verified |
firefox70 | --- | verified |
People
(Reporter: muirpablo, Assigned: emcminn)
References
(Regression)
Details
(Keywords: github-merged, regression)
Attachments
(3 files)
(deleted),
text/x-github-pull-request
|
Details | |
(deleted),
text/x-github-pull-request
|
Details | |
(deleted),
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details |
[Description:]
The PIN option is always highlighted in the Top sites area.
[Environment:]
69.0a1 Nightly Win10 / Ubuntu 18.04 / Macos 10.14.4 (all affected)
68.0b14 Beta Win10 no/ Ubuntu 18.04 no/ Macos 10.14.4 (not affected)
67.0.4 Release Win10 no/ Ubuntu 18.04 no/ Macos 10.14.4 (not affected)
[Steps:]
Launch firefox
Open new tab
Check Top Sites
Click on "..." on one of the top sites
See "Pin" option
[Actual Result:]
Pin option is always highlighted, even if you hover over another option
[Expected Result:]
Pin option should not be highlighted, it should be highlighted only if you hover over it.
[Notes:]
Mozregression:
changeset: c04923179a923f5a94af9b6df475c266adbf39f0
pushlog_url: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=c04923179a923f5a94af9b6df475c266adbf39f0&tochange=a72f0a08f652ab309328230659c97660e13cfd27
Note this issue not only affects Top Sites, it also happens on the "Recommended by Pocket" section. As the "bookmark" option is also always selected once you click on "..."
Comment 2•5 years ago
|
||
Reverting bug 1559383 seems to fix the problem locally for me. It's just that opening the menu focuses the item, so pointing with the mouse also shows a hover effect too in addition to the focus.
Assignee | ||
Comment 3•5 years ago
|
||
To me it looks like the issue is that if the user switches keyboard and mouse, the keyboard highlight remains when the mouse highlights another option. Should be able to fix this by allowing only one option to be focused at a time. I'll take a look!
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 4•5 years ago
|
||
So we've identified a way to fix this but it'll take a pretty hefty refactor - for now I'll remove the focus behavior entirely, and fix both that and the highlighting with a future patch.
Comment 5•5 years ago
|
||
Comment 6•5 years ago
|
||
Comment 7•5 years ago
|
||
Comment 8•5 years ago
|
||
I have verified that the issue is no longer reproducible on the latest Nightly 70.0a1 (Build ID 20190713215744) on Windows 10, macOS 10.14, and Arch Linux 14.4.3.
@Ed, given that bug 1559383 has been reverted, should we reopen it in order to track the issue moving forward?
Comment 9•5 years ago
|
||
Sounds reasonable. There's already a PR to fix things https://github.com/mozilla/activity-stream/pull/5161 I'll update it to point to the reopened bug
Updated•5 years ago
|
Comment 10•5 years ago
|
||
Is this something we should consider for Beta uplift for Fx69?
Assignee | ||
Comment 11•5 years ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #10)
Is this something we should consider for Beta uplift for Fx69?
I think so!
Comment 12•5 years ago
|
||
OK, just waiting on a patch and approval request :)
Comment 13•5 years ago
|
||
Assignee | ||
Comment 14•5 years ago
|
||
Comment on attachment 9081306 [details]
Bug 1562894 - Top sites PIN option is always highlighted
Beta/Release Uplift Approval Request
- User impact if declined: Focus behavior of context menus will be incorrect.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: See steps in https://bugzilla.mozilla.org/show_bug.cgi?id=1562894
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This patch backs out a change to focus behavior and reverts it to the previous correct behavior.
- String changes made/needed: none
Assignee | ||
Updated•5 years ago
|
Comment 15•5 years ago
|
||
Comment on attachment 9081306 [details]
Bug 1562894 - Top sites PIN option is always highlighted
Thanks for the approval request. Approved for 69.0b11.
Comment 16•5 years ago
|
||
Ryan, I grafted the patch here and another bug got in, bug 1565293: https://irccloud.mozilla.com/file/ABNAAZp4/image.png
I backed that bug out, and landed this one. Sorry for the mess.
Comment 17•5 years ago
|
||
bugherder uplift |
Updated•5 years ago
|
Updated•5 years ago
|
Comment 18•5 years ago
|
||
I have verified that this issue is no longer reproducible with the latest Firefox Beta (69.0b14 Build ID - 20190815163925) installed, on Windows 10 x64, Arch Linux and Mac 10.14.5. Now the first option from the menu is no longer highlighted by default.
Updated•3 years ago
|
Description
•