898294 - [Gallery] Statusbar show when switching from camera app to gallery

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Gallery, defect)

Description

[(Inactive after June) George Duan [:gduan] [:喬智]]

Attachment: gallery.png

## Environment
Gaia: ccf877b99ace72e621eb5172f042c6c20aa1297a

STR:
1. Open gallery app
2. click one picture
3. click camera icon on the bottom bar, then switch to camera app
4. click left-bottom icon, then switch to gallery app
5. click edit button (center icon on the bottom)

Expect: statusbar should not show
Actual: statusnar show and overlap header

see attachment.

Comment 1

[Wayne Chang [:wchang]]

Triage - Small user impact, no functional impact, polish bug. Leo-

Comment 2

[(Inactive after June) George Duan [:gduan] [:喬智]]

Hi Alive,
I found this bug is from bug 896393.
When animation ends after opening inline window, window_manager will trigger |setCloseFrame(null)|, and then |windowClosed| remove fullscreen-app class from screen. 

I'm thinking to remove |screenElement.classList.remove('fullscreen-app');| from |windowClosed|, since it already be triggered by |setDisplayedApp| before switching/opening/closing app. 

What do you think?

Comment 3

[(inactive after 6/18) Alive Kuo [:alive]]

Device is not by my side.
IMO clear fullscreen state after window is closed is correct.
So what shall be done is add fullscreen back when the opening app is fullscreen.

Be careful to call windowClosed before openWindow or add the fullscreen state again after windowOpened.

Comment 4

[(Inactive after June) George Duan [:gduan] [:喬智]]

Attachment: PR to master

This patch add fullscreen-app class to displayed app when invoked/invoking animation is done

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 784317 [details]
PR to master

Ignore my previous comment, your original proposal is fine. Use that. Thanks.

Comment 6

[(Inactive after June) George Duan [:gduan] [:喬智]]

Thanks Alive!

Merge into master:
https://github.com/mozilla-b2g/gaia/commit/a8b0a40337b91bfa82795efc36d6defffa2eda23