Unified Diff: lib/filterComposer.js
Issue 29370947:
Issue 3138 - Improve how context menu "block element" handles iframes (Closed)
Patch Set: Addressed feedback
Created Jan. 12, 2017, 7:14 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« composer.postload.js ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/filterComposer.js |
| diff --git a/lib/filterComposer.js b/lib/filterComposer.js |
| index 10390f9f3be7462ec52da3639872fa1df757b2a8..8f1494e70e8f6fc64041dc53ebf7cc996aceb1e1 100644 |
| --- a/lib/filterComposer.js |
| +++ b/lib/filterComposer.js |
| @@ -156,9 +156,11 @@ function composeFilters(details) |
| let contextMenuItem = { |
| title: ext.i18n.getMessage("block_element"), |
| contexts: ["image", "video", "audio"], |
| - onclick: page => |
| + onclick: (page, info) => |
| { |
| - page.sendMessage({type: "composer.content.contextMenuClicked"}); |
| + page.sendMessage( |
| + {type: "composer.content.contextMenuClicked"}, undefined, info.frameId |
| + ); |
| } |
| }; |
« no previous file with comments
|
« composer.postload.js ('k')
|
no next file »
|
no next file with comments »
