Firefox OS v1.3 Mozilla build ID:20140422024003 Created an attachment (id=706750) pic DEFECT DESCRIPTION: [SMS][Gallery]The short message shouldn't be opened under the picture share interface when open a new sms via notifications in status bar. REPRODUCING PROCEDURES: 1.Launch Gallery,view a picture,tap the 'share' button,pop up share interface 2.A SMS arrive,tap the notification in the status bar,you can see the SMS opened below the PIC share interface-->K.O EXPECTED BEHAVIOUR: MS should enter SMS interface when tap message notification ASSOCIATE SPECIFICATION: TEST PLAN REFERENCE: TOOLS AND PLATFORMS USED: USER IMPACT: Medium REPRODUCING RATE: 5/5 For FT PR, Please list reference mobile's behavior: Beetle lite FF V18H can reproduce this bug Soul 3.5 FF V121+ZZ can reproduce this bug

I think this has been fixed with bug 936729. Can you try with a newer build on master?

(In reply to Julien Wajsberg [:julienw] from comment #3) > I think this has been fixed with bug 936729. > > Can you try with a newer build on master? We just have a 20140422024003 build now. Thus let's wait for the new build upgraded.

(In reply to 田旻 from comment #4) > (In reply to Julien Wajsberg [:julienw] from comment #3) > > I think this has been fixed with bug 936729. > > > > Can you try with a newer build on master? > > We just have a 20140422024003 build now. Thus let's wait for the new build > upgraded. Ya i dont think we will ever uplift the patch of 936729, so Tianm you might need to cherry-pick the fix yourself

(In reply to Vance Chen [:vchen][vchen@mozilla.com] from comment #5) > (In reply to 田旻 from comment #4) > > (In reply to Julien Wajsberg [:julienw] from comment #3) > > > I think this has been fixed with bug 936729. > > > > > > Can you try with a newer build on master? > > > > We just have a 20140422024003 build now. Thus let's wait for the new build > > upgraded. > > Ya i dont think we will ever uplift the patch of 936729, so Tianm you might > need to cherry-pick the fix yourself OK, I will have a try.

(In reply to Julien Wajsberg [:julienw] (away until 5th May) from comment #3) > I think this has been fixed with bug 936729. > > Can you try with a newer build on master? But they are not the same bug, this one is just about the covering of the shared interface to the new opened sms.

(In reply to Julien Wajsberg [:julienw] (away until 5th May) from comment #3) > I think this has been fixed with bug 936729. > > Can you try with a newer build on master? Could you kind enough and have a double check? Thanks.

(In reply to 田旻 from comment #7) > (In reply to Julien Wajsberg [:julienw] (away until 5th May) from comment #3) > > I think this has been fixed with bug 936729. > > > > Can you try with a newer build on master? > > But they are not the same bug, this one is just about the covering of the > shared interface to the new opened sms. Ok, I think I was confused; in the step 1, you don't choose any application ? Then it's really how activities and system messages work. When you click "share", the system app shows the "applications" menu. Then you click on the notification, and the system app runs the SMS app. I'll redirect this bug to the System app because it's a Window Manager issue. Alive will probably be able to tell you more about this.

We should dismiss the action menu if any app is opened. This is a simple fix.

(In reply to Julien Wajsberg [:julienw] (away until 5th May) from comment #9) > (In reply to 田旻 from comment #7) > > (In reply to Julien Wajsberg [:julienw] (away until 5th May) from comment #3) > > > I think this has been fixed with bug 936729. > > > > > > Can you try with a newer build on master? > > > > But they are not the same bug, this one is just about the covering of the > > shared interface to the new opened sms. > > Ok, I think I was confused; in the step 1, you don't choose any application ? > > Then it's really how activities and system messages work. When you click > "share", the system app shows the "applications" menu. Then you click on the > notification, and the system app runs the SMS app. > > I'll redirect this bug to the System app because it's a Window Manager > issue. Alive will probably be able to tell you more about this. Yes, you are right.

This doesn't feel like something we would hold the release on - it's not high impact if this occurs, so this is not a blocker.

I'd like to steal it if Alive doesn't mind :p

Comment on attachment 8426002 [details] PR to master Thanks, but I prefer to let activities to do that (hide menu on opened) for action menu because if we have multiple action menus in the future, the event handler may cause trouble to close some menus we don't want to close.

Comment on attachment 8426002 [details] PR to master Thank you!

Travis is green https://travis-ci.org/mozilla-b2g/gaia/builds/25840814 landed on master https://github.com/mozilla-b2g/gaia/commit/01aa9c4bb724aba0ec254738000a2d506068f0c5