Unified Diff: include.postload.js
Issue 5838948538515456:
Issue 370 - Make "Block element" hide elements for added filters (Closed)
Patch Set: Replace existing rules
Created March 4, 2015, 9:10 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« block.js ('K')
|
« block.js ('k')
|
include.preload.js » ('j')
|
include.preload.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: include.postload.js |
| =================================================================== |
| --- a/include.postload.js |
| +++ b/include.postload.js |
| @@ -707,10 +707,15 @@ |
| case "clickhide-close": |
| if (currentElement && msg.remove) |
| { |
| - // Explicitly get rid of currentElement |
| - var element = currentElement.prisoner || currentElement; |
| - if (element && element.parentNode) |
| - element.parentNode.removeChild(element); |
| + // Hide the selected element itself if an added blocking |
| + // filter is causing it to collapse. Note that this |
| + // behavior is incomplete, but the best we can do here, |
| + // e.g. if an added blocking filter matches other elements, |
| + // the effect won't be visible until the page is is reloaded. |
| + checkCollapse(currentElement.prisoner || currentElement); |
| + |
| + // Apply added element hiding filters. |
| + updateStylesheet(); |
| } |
| clickHide_deactivate(); |
| break; |
