In Firefox Nightly with a DEBUG profile, click the everything.me bar. Now there is mayhem as you can't use the homescreen anymore.
Created attachment 8370666 [details] [review] Patch So what happens on FF Nightly is that the templates for the dialogs get filled when you click everything.me bar. I don't really know why that happens. The problem is that all the styling for these elements are in CSS files that will be lazy loaded when needed, causing the elements to show up with their default styles, which is why they are visible. There is no JS logic behind it either (also lazy loaded) so you can't dismiss it. This patch adds a default style of display: none to the dialogs in homescreen.css that will be overwritten when we load the dialog CSS on demand. Tested it on the device as well and the context menu's still work as expected.
Comment on attachment 8370666 [details] [review] Patch Awesome, thanks for doing this!
Attachment #8370666 - Flags: feedback?(kgrandon) → feedback+
Honestly, it should not happen because both are elements that haven't imported yet. So this patch works perfectly but it could hide the root issue thought https://github.com/mozilla-b2g/gaia/blob/master/apps/homescreen/index.html#L53
Yeah, my assumption was that E.me was trying to load some CSS but crashing/failing somewhere before it does.. It would probably be good to have an understanding of the root issue before landing.
Maybe Ran can have any idea
Ran, any thought? Thx mate
Amir, do you?
Flags: needinfo?(ran) → needinfo?(amirn)
Does not happen on device, I always assumed this is a Nightly issue. Looks like lazy_loader.js behaves differently when DBEUG=1: https://github.com/mozilla-b2g/gaia/blame/master/shared/js/lazy_loader.js#L43 Kevin?
Flags: needinfo?(amirn) → needinfo?(kgrandon)
Yes, it behaves differently - but I don't think it should cause the issues shown. Instead of inlining elements, we just lazily load them. It sounds like we need to just dig in to see exactly what's going on here.
Well, anyway, we're having a bunch of div's that have a display: block state whereas they should not be visible at all in default state (regardless of whether there is content in them) so we could land this anyway. Then we can take a look at the reason why we populate the divs when we focus on the e.me bar.
Attachment #8370666 - Flags: review?(21) → review?(crdlc)
Comment on attachment 8370666 [details] [review] Patch OK, delete dialog and context menu work fine, thanks
Attachment #8370666 - Flags: review?(crdlc) → review+
Thanks Christian, https://github.com/mozilla-b2g/gaia/commit/e52cf841dc791f8c00748cd49c8e87d69f506041
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.