846422 - Change - Context menu overlays flyouts

Status: VERIFIED FIXED

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

Description

[Jim Mathies [:jimm]]

Attachment: screen cap

See screen shot. We need to move these down in the stack.

Comment 1

[Marco Mucci [:MarcoM]]

Which story or work item does this block Jim?

Comment 2

[Jim Mathies [:jimm]]

Attachment: move flyouts

Flyouts should be at the bottom of pretty much everything I think.

Comment 3

[Jim Mathies [:jimm]]

That patch sucks since it blows away blame. All I've really done is copy flyouts down to the bottom of the stack.

Comment 4

[Jim Mathies [:jimm]]

(In reply to Marco Mucci [:MarcoM] from comment #1)
> Which story or work item does this block Jim?

Hmm, something flyouts related maybe?

Comment 5

[Matt Brubeck (:mbrubeck)]

Comment on attachment 719612 [details] [diff] [review]
move flyouts

We need menus to be able to appear over flyoutpanels; for example we use a MenuPopup for the menulist control in the "Startup" section of the Options flyout.  It seems likely that we will also want text selection or context menus within flyoutpanels at some point.

For the case in your screenshot, maybe we should get better about hiding the flyoutpanel when interacting with content (and/or hiding the context menu when showing the flyoutpanel).

Comment 6

[Jim Mathies [:jimm]]

Comment on attachment 719612 [details] [diff] [review]
move flyouts

ah poop. thought it was an easy fix.

Comment 7

[Asa Dotzler [:asa]]

I think we should be clearing any context menus when the user swipes in the charms bar.

Comment 8

[Rodrigo Silveira [:rsilveira] [:rodms]]

Attachment: Patch v1

Comment 9

[Matt Brubeck (:mbrubeck)]

Comment on attachment 730251 [details] [diff] [review]
Patch v1

https://hg.mozilla.org/integration/mozilla-inbound/rev/4afb61ad2964

Comment 10

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/4afb61ad2964

Comment 11

[Marco Mucci [:MarcoM]]

Just reopening for inclusion into Iteration #5.

Comment 12

[Virgil Dicu [:virgil] [QA]]

Mozilla/5.0 (Windows NT 6.2; WOW64; rv:23.0) Gecko/20130404 Firefox/23.0

Verified that context menu, for a selected text on a page, is not accessible when the user swipes in the charms bar and selects an option.

Comment 13

[Kamil Jozwiak [:kjozwiak]]

Hi Jim, added the issues that I found here as the original "Story" seems to be missing

Comment 14

[Jim Mathies [:jimm]]

I don't think we need to block in this. Each bug is different.

Comment 15

[Kamil Jozwiak [:kjozwiak]]

Went through the following "Defect" for iteration #7 testing without any issues. Used the following build:

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/2013-05-28-03-09-42-mozilla-central/

- Ensured that selecting "Options", "Sync" or "About" clears the "Context Menu" without any issues
- Ensured that both "Link" & "Image" Context Menu's are dismissed when selecting "Options", "Sync" or "About"
- Ensured that the "Context Menu" is being cleared when creating a new tab using the junior style button
- Ensured that the "Context Menu" is being cleared when creating a new tab using the "+" button under the "Navigation App Bar"
- Ensured that the "Context Menu" is being cleared when selecting the junior style "Back" button
- Ensured that the "Context Menu" is being cleared when sliding in the "Navigation App Bar" from both the bottom & top

Issues still occurring with the Context Menu:

- Bug 861536 (Defect) still an an issue (reproduced several times)
- Bug 861532 (Defect) is still an issue (reproduced several times)