Closed Bug 1235470 Opened 7 years ago Closed 7 years ago
Move "z-index: 2147483647 !important" for :-moz-full-screen from ua
.css to gaia code
This z-index is no longer necessary for fullscreen as we now render almost all fullscreen elements in the top layer (bug 1126230). The only exceptions are :root and <iframe mozbrowser>. It seems there have been various hack in gaia code to promote system UI on top of fullscreen element, which we could probably get rid of now, because it no longer makes sense to keep the "important" z-index setting in ua.css. But getting rid of that would need conjunct work between gecko code and gaia code, which I have no idea how to. A potential first step is to add a rule in gaia/apps/system/style/zindex.css to keep the current setting of fullscreen element, then remove the line from ua.css. And after that, remaining clean-up work could happen inside gaia code without touching the gecko code.
Attachment #8702429 - Flags: review?(etienne)
Comment on attachment 8702429 [details] [review] [gaia] upsuper:bug1235470 > mozilla-b2g:master stamp :)
Attachment #8702429 - Flags: review?(etienne) → review+
If the gaia code gets landed, I think I can reland my patch in bug 1215365 directly then.
Summary: Remove "z-index: 2147483647 !important" for :-moz-full-screen from ua.css → Move "z-index: 2147483647 !important" for :-moz-full-screen from ua.css to gaia code
Could you merge that pull request for me? I don't have write access to b2g repos.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.