Closed Bug 927239 Opened 6 years ago Closed 6 years ago

Defect - Context menu items keep highlight after being selected

Categories

(Firefox for Metro Graveyard :: Input, defect, P2)

x86_64
Windows 8.1
defect

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 28

People

(Reporter: kjozwiak, Assigned: sfoster)

References

Details

(Whiteboard: [block28] feature=defect c=context_menus u=metro_firefox_user p=1)

Attachments

(3 files, 1 obsolete file)

If a user taps & holds either an image or a link and receives the context menu, the selection will stay highlighted if the user taps, holds and then moves away from the context menu. If you dismiss and then get another context menu, you'll notice that the selection is still highlighted from the previous selection.

- Attached a screenshot to illustrate the issue (item was selected with a new context menu)

Steps to reproduce the issue:

1) Open Firefox Metro
2) Go to wikipedia.org
3) Get a context menu by either tapping on a link or image
4) Tap & hold a context menu item and it will be highlighted black, while still having your finger on the screen, move away from the context menu and let go (notice that the item is still highlighted)
5) Dismiss the context menu and then tap and hold another link. You should see that the one of the items in the context menu will still be highlighted from the last time

Current Behavior:

- items under the context menu stay highlighted if you tap, hold and then slide your finger away from the context menu. The user will still have an item highlighted when they dismiss & get new context menu

Expected Behavior:

- items should be cleared when the context menu is dismissed. When a new context menu appears, it should be cleared of any previous selections
Whiteboard: feature=defect c=context_menus u=metro_firefox_user p=0 → [triage] feature=defect c=context_menus u=metro_firefox_user p=0
I'm not able to reproduce this. First, using touch (not mouse) I see no highlight states in the context menu, when I tap+hold on a link or image. Perhaps this is a bug. 
Using the mouse, the black highlight seems to revert correctly when the menu is dismissed. I'm using a Lenovo X1 Carbon which I think is the same machine as :kjozwiak is using?
I went through it and reproduced the issue with today's build, I added a small video on how to reproduce the problem. When the context menu appears, I tap, hold and then swipe away from the context menu and then let go (the item will stay highlighted).

Hope the video helps! let me know if you still cant reproduce it.
Ok, now I see the highlight and the behavior you describe. Thanks, I'll look into it.
Whiteboard: [triage] feature=defect c=context_menus u=metro_firefox_user p=0 → feature=defect c=context_menus u=metro_firefox_user p=0
If it helps at all, I noticed that I'm only able to reproduce this bug with apzc on.
Blocks: metro-apzc
Summary: Defect - Context menu's stay highlighted → Defect - Context menu items keep highlight after being selected
This appears to be fixed on latest tip. Kamil, mind confirming?

Note we still have a lost tap / touch problem to deal with. Sometimes taps just don't get picked up on the menu and the context stays open. I think this is related to apz. It's also filed under a different bug.
Flags: needinfo?(kamiljoz)
Assignee: nobody → jmathies
Whiteboard: feature=defect c=context_menus u=metro_firefox_user p=0 → [blocker] feature=defect c=context_menus u=metro_firefox_user p=0
Hey Jim, can you provide a point estimate.
Blocks: metrov1it18
No longer blocks: metrov1backlog
Status: NEW → ASSIGNED
Flags: needinfo?(jmathies)
Priority: -- → P2
QA Contact: jbecerra
Whiteboard: [blocker] feature=defect c=context_menus u=metro_firefox_user p=0 → [block28] feature=defect c=context_menus u=metro_firefox_user p=0
Flags: needinfo?(jmathies)
Whiteboard: [block28] feature=defect c=context_menus u=metro_firefox_user p=0 → [block28] feature=defect c=context_menus u=metro_firefox_user p=1
Jim,

I went through the issue again with the latest nightly and could still reproduce the problem on several websites, used the following build: (used the same steps from comment #0)
http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-10-28-03-02-05-mozilla-central/

Please let me know if there's another build that I should have tested with!
Flags: needinfo?(kamiljoz)
fixed in 10-31 nightly.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 28
I'm still seeing this with these steps: 

* Load any page
* Right-click or long-tap a link on the page to bring up the link context menu
* Select open in new tab (or any of the options)
* Right-click again on the link (or any link) to re-open the context menu

Expected results: 
* None of the context menu options should be pre-selected when the menu shows

Actual results: 
* The option previously selected when the context menu was last opened is selected (highlighted orange)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Not sure if the earlier issue was just masking this one, or if this actually belongs in its own bug. However, afaict hide() is the right place to reset the state of those commands/richlistitems.
Attachment #829537 - Flags: review?(jmathies)
sigh. Patch clean-up, missed a qrefresh.
Assignee: jmathies → sfoster
Attachment #829537 - Attachment is obsolete: true
Attachment #829537 - Flags: review?(jmathies)
Attachment #829544 - Flags: review?(jmathies)
Blocks: metrov1it19
No longer blocks: metrov1it18
Comment on attachment 829544 [details] [diff] [review]
remove selected attribute from menu items when hiding context menu

wfm!
Attachment #829544 - Flags: review?(jmathies) → review+
selected attribute patch landed on fx-team https://hg.mozilla.org/integration/fx-team/rev/ea7a81a92657

there's still an event-related issue though, if you tap a menu item then drag away before dismissing the menu the :hover state never gets cleared, so the item is highlighted grey when the menu is next opened.
Whiteboard: [block28] feature=defect c=context_menus u=metro_firefox_user p=1 → [block28] feature=defect c=context_menus u=metro_firefox_user p=1 [leave-open]
Whiteboard: [block28] feature=defect c=context_menus u=metro_firefox_user p=1 [leave-open] → [block28] feature=defect c=context_menus u=metro_firefox_user p=1 [leave open]
As Sam mentioned in comment #13, there's still an event issue if you follow his steps or the steps in comment #0. I can still reproduce the issue using the latest nightly build:
http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-11-10-03-02-05-mozilla-central/
(In reply to Kamil Jozwiak [:kjozwiak] from comment #15)
> As Sam mentioned in comment #13, there's still an event issue if you follow
> his steps or the steps in comment #0. I can still reproduce the issue using
> the latest nightly build:
> http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-11-10-03-02-05-
> mozilla-central/

This seems like a corner case - you have to slide your finger off the context menu to make this happen.
No longer blocks: metro-apzc
Whiteboard: [block28] feature=defect c=context_menus u=metro_firefox_user p=1 [leave open] → [triage] feature=defect c=context_menus u=metro_firefox_user p=1 [leave open]
Follow-up bug filed for the :hover issue at bug 937229.
Status: REOPENED → RESOLVED
Closed: 6 years ago6 years ago
Resolution: --- → FIXED
Whiteboard: [triage] feature=defect c=context_menus u=metro_firefox_user p=1 [leave open] → [triage] feature=defect c=context_menus u=metro_firefox_user p=1
Whiteboard: [triage] feature=defect c=context_menus u=metro_firefox_user p=1 → [block28] feature=defect c=context_menus u=metro_firefox_user p=1
Verified as fixed for iteration #19, with latest Nightly from 2013-12-01 on Win 8 64-bit, by following the STR from comment 9.

This also needs device verification.
Status: RESOLVED → VERIFIED
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.