Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/filterComposer.js

Issue 29901596: Issue 6493 - Ensure Firefox draws the "Block element" popup window (Closed)
Patch Set: Avoid hardcoding the width again Created Oct. 8, 2018, 2:48 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/filterComposer.js
diff --git a/lib/filterComposer.js b/lib/filterComposer.js
index fe5dcdf3071e1762e31cfb4e4bd461a67bbcac23..6e48bd48385f1f1748997ff7ceaac1ee0814d9a0 100644
--- a/lib/filterComposer.js
+++ b/lib/filterComposer.js
@@ -188,6 +188,11 @@ port.on("composer.openDialog", (message, sender) =>
// we must check the response too.
if (!response)
throw new Error();
+
+ // Sometimes Firefox doesn't draw the window's contents initially, so we
+ // resize the window slightly as a workaround.
+ // https://issues.adblockplus.org/ticket/6493
+ browser.windows.update(window.id, {width: window.width + 2});
Sebastian Noack 2018/10/08 15:11:53 Still the window ends up 2 pixels larger as it mea
kzar 2018/10/08 15:58:28 Well, I guess so, but I really don't think that ma
Sebastian Noack 2018/10/08 16:07:22 I think this workaround should not change the effe
kzar 2018/10/08 17:23:15 Done.
}).catch(e =>
{
// Firefox sometimes sets the status for a window to "complete" before
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld