1369312 - Multiple notifications are overlapped while tapping power key

Status: RESOLVED INVALID

(Firefox for Android Graveyard :: General, defect, P3)

Description

[Al Tsai [:atsai]]

STR:
1. Go to a webpage with Reader mode
2. tap on power key to turn off the screen
3. tap on power key to turn on the screen
4. repeat step 2, and 3 for multiple times
5. tap cancel to dismiss the dialog

Expected Results:
1. Web is loaded w/ a dialog for using Reader mode
2. screen is off
3. screen is on and the screen is the same as the result of step 1
4. dialog dismissed

Actual Results:
4. multiple dialog on the screen

We'll need a mechanism to check how many dialogs on the screen. Ideally, there should be one at most at a time. bug 1367314 defined some guidelines but didn't include such scenario. Tori is aware of it.

Comment 1

[Joe Cheng [:jcheng] (please needinfo)]

given how the in app message is displayed in the initial implementation (unless we can make the HTML5 message work), we should not promote this use case. making this is P2

Comment 2

[Nevin Chen(Not active on Bugzilla)]

This is the expected behavior. It just a server-sdie setting. I set it to be triggered everytime just for easy testing.
Please try it again since I've changed the server setting to how it's really used.

Comment 3

[Al Tsai [:atsai]]

I understood that this is triggered by server-side. However, it causes users get confused while they attempt to close/remove the dialog. We'll need a solution for it. Inputs from UX can be valuable.

Comment 4

[Tori Chen [:tori]]

Showing same dialog after just closing one is really weird and we should avoid this case. I suggest that for one promotion message, the old dialog should be replaced once there is a new one.

Comment 5

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

this is already handled from the server-side configuration, so this is no longer reproducible.
in the long run we may want some client-side fix so I'm leaving this open but P3.

Comment 6

[Nevin Chen(Not active on Bugzilla)]

This is fixed. Thanks!

Comment 7

[Nevin Chen(Not active on Bugzilla)]

We have disabled this message so we can close the bug.