From 89df06fa6c00c9295ff064308c03f770b01d79be Mon Sep 17 00:00:00 2001 From: Stephen Blott Date: Sat, 2 Apr 2016 11:54:04 +0100 Subject: Rework DomUtils.documentReady(). There seems to be an issue on this page: - http://i.imgur.com/PdmUjij.jpg whereby "DOMContentLoaded" isn't firing. The page structure is unusual (involving a shadow-DOM element), which might be the source of the problem. What happens is that the "DOMContentLoaded" event fires as required, but the document state is still "loading". Here, we just say that if the "DOMContentLoaded" handler has fired once, then we're good to go. --- content_scripts/vimium_frontend.coffee | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'content_scripts') diff --git a/content_scripts/vimium_frontend.coffee b/content_scripts/vimium_frontend.coffee index f14c5d07..39bfaacb 100644 --- a/content_scripts/vimium_frontend.coffee +++ b/content_scripts/vimium_frontend.coffee @@ -436,10 +436,10 @@ extend window, # Initialize UI components which are only installed in the main/top frame. initializeTopFrameUIComponents = do -> - Frame.addEventListener "initializeTopFrameUIComponents", Utils.makeIdempotent Vomnibar.init.bind Vomnibar + Frame.addEventListener "initializeTopFrameUIComponents", Utils.makeIdempotent -> + DomUtils.documentReady Vomnibar.init.bind Vomnibar - Utils.makeIdempotent -> - DomUtils.documentReady -> Frame.postMessage "initializeTopFrameUIComponents" + Utils.makeIdempotent -> Frame.postMessage "initializeTopFrameUIComponents" # Initialize UI components which are only installed in all frames (i.e., the HUD). initializeAllFrameUIComponents = Utils.makeIdempotent -> -- cgit v1.2.3