fix #1772: ability to preview procedural cosmetic filters

platform/chromium/vapi-client.js

@@ -142,25 +142,24 @@ vAPI.setTimeout = vAPI.setTimeout || self.setTimeout.bind(self);
 
 /******************************************************************************/
 
-vAPI.shutdown = (function() {
+vAPI.shutdown = {
-    var jobs = [];
+    jobs: [],
-
+    add: function(job) {
-    var add = function(job) {
+        this.jobs.push(job);
-        jobs.push(job);
+    },
-    };
+    exec: function() {
-
-    var exec = function() {
         var job;
-        while ( (job = jobs.pop()) ) {
+        while ( (job = this.jobs.pop()) ) {
             job();
         }
-    };
+    },
-
+    remove: function(job) {
-    return {
+        var pos;
-        add: add,
+        while ( (pos = this.jobs.indexOf(job)) !== -1 ) {
-        exec: exec
+            this.jobs.splice(pos, 1);
-    };
+        }
-})();
+    }
+};
 
 /******************************************************************************/
 /******************************************************************************/

platform/firefox/vapi-client.js

@@ -123,26 +123,24 @@ vAPI.setTimeout = vAPI.setTimeout || function(callback, delay, extra) {
 
 /******************************************************************************/
 
-vAPI.shutdown = (function() {
+vAPI.shutdown = {
-    var jobs = [];
+    jobs: [],
-
+    add: function(job) {
-    var add = function(job) {
+        this.jobs.push(job);
-        jobs.push(job);
+    },
-    };
+    exec: function() {
-
-    var exec = function() {
-        //console.debug('Shutting down...');
         var job;
-        while ( (job = jobs.pop()) ) {
+        while ( (job = this.jobs.pop()) ) {
             job();
         }
-    };
+    },
-
+    remove: function(job) {
-    return {
+        var pos;
-        add: add,
+        while ( (pos = this.jobs.indexOf(job)) !== -1 ) {
-        exec: exec
+            this.jobs.splice(pos, 1);
-    };
+        }
-})();
+    }
+};
 
 /******************************************************************************/
 

src/epicker.html

@@ -56,12 +56,14 @@ section {
     border: 0;
     box-sizing: border-box;
     display: inline-block;
-    position: relative;
     width: 100%;
 }
-section > textarea {
+section > div {
+    position: relative;
+}
+section > div > textarea {
     background-color: #fff;
-    border: 1px solid #ccc;
+    border: 1px solid #aaa;
     box-sizing: border-box;
     font: 11px monospace;
     height: 6em;
@@ -70,15 +72,22 @@ section > textarea {
     resize: none;
     width: 100%;
 }
-section > div {
+section > div > textarea.invalidFilter {
+    background-color: #fee;
+}
+section > div > textarea + div {
+    background-color: #aaa;
+    bottom: 0;
+    color: white;
+    padding: 2px 4px;
+    position: absolute;
+    right: 0;
+}
+section > div + div {
     direction: ltr;
     margin: 2px 0;
     text-align: right;
 }
-section > div > span:last-of-type {
-    position: absolute;
-    right: 0;
-}
 ul {
     padding: 0;
     list-style-type: none;
@@ -137,8 +146,12 @@ svg > path + path {
 body.preview svg > path {
     fill: rgba(0,0,0,0.10);
 }
+body.preview svg > path + path {
+    stroke: none;
+}
 aside {
     background-color: #eee;
+    border: 1px solid #aaa;
     bottom: 4px;
     box-sizing: border-box;
     visibility: hidden;
@@ -162,7 +175,10 @@ body.paused > aside:hover {
 <svg><path></path><path></path></svg>
 <aside>
 <section>
-    <textarea lang="en" dir="ltr" spellcheck="false"></textarea>
+    <div>
+        <textarea lang="en" dir="ltr" spellcheck="false"></textarea>
+        <div></div>
+    </div>
     <div><!--
     --><button id="preview" type="button">{{preview}}</button><!--
     --><button id="create" type="button" disabled>{{create}}</button><!--

src/js/scriptlets/element-picker.js

@@ -21,14 +21,14 @@
 
 /* global CSS */
 
+'use strict';
+
 /******************************************************************************/
 /******************************************************************************/
 
 /*! http://mths.be/cssescape v0.2.1 by @mathias | MIT license */
 ;(function(root) {
 
-    'use strict';
-
     if (!root.CSS) {
         root.CSS = {};
     }
@@ -116,8 +116,6 @@
 
 (function() {
 
-'use strict';
-
 /******************************************************************************/
 
 if ( typeof vAPI !== 'object' ) {
@@ -147,7 +145,6 @@ var cosmeticFilterCandidates = [];
 var targetElements = [];
 var candidateElements = [];
 var bestCandidateFilter = null;
-var previewedElements = [];
 
 var lastNetFilterSession = window.location.host + window.location.pathname;
 var lastNetFilterHostname = '';
@@ -268,65 +265,6 @@ var highlightElements = function(elems, force) {
 
 /******************************************************************************/
 
-var filterElements = function(filter) {
-    var htmlElem = document.documentElement;
-    var items = elementsFromFilter(filter);
-    var i = items.length, item, elem, style;
-    while ( i-- ) {
-        item = items[i];
-        elem = item.elem;
-        // https://github.com/gorhill/uBlock/issues/1629
-        if ( elem === pickerRoot ) {
-            continue;
-        }
-        style = elem.style;
-        if (
-            (elem !== htmlElem) &&
-            (item.type === 'cosmetic' ||
-             item.type === 'network' && item.src !== undefined)
-        ) {
-            previewedElements.push({
-                elem: elem,
-                prop: 'display',
-                value: style.getPropertyValue('display'),
-                priority: style.getPropertyPriority('display')
-            });
-            style.setProperty('display', 'none', 'important');
-        }
-        if ( item.type === 'network' && item.style === 'background-image' ) {
-            previewedElements.push({
-                elem: elem,
-                prop: 'background-image',
-                value: style.getPropertyValue('background-image'),
-                priority: style.getPropertyPriority('background-image')
-            });
-            style.setProperty('background-image', 'none', 'important');
-        }
-    }
-};
-
-/******************************************************************************/
-
-var preview = function(filter) {
-    filterElements(filter);
-    pickerBody.classList.add('preview');
-};
-
-/******************************************************************************/
-
-var unpreview = function() {
-    var items = previewedElements;
-    var i = items.length, item;
-    while ( i-- ) {
-        item = items[i];
-        item.elem.style.setProperty(item.prop, item.value, item.priority);
-    }
-    previewedElements.length = 0;
-    pickerBody.classList.remove('preview');
-};
-
-/******************************************************************************/
-
 // https://github.com/gorhill/uBlock/issues/1897
 // Ignore `data:` URI, they can't be handled by an HTTP observer.
 
@@ -692,117 +630,373 @@ var filtersFrom = function(x, y) {
     return netFilterCandidates.length + cosmeticFilterCandidates.length;
 };
 
-/******************************************************************************/
+/*******************************************************************************
 
-var elementsFromFilter = function(filter) {
+    filterToDOMInterface.set
-    var out = [];
+    @desc   Look-up all the HTML elements matching the filter passed in
+            argument.
+    @param  string, a cosmetic of network filter.
+    @return array, or undefined if the filter is invalid.
 
-    filter = filter.trim();
+    filterToDOMInterface.preview
-    if ( filter === '' ) {
+    @desc   Apply/unapply filter to the DOM.
-        return out;
+    @param  string, a cosmetic of network filter, or literal false to remove
-    }
+            the effects of the filter on the DOM.
+    @return undefined.
 
-    // Cosmetic filters: these are straight CSS selectors
+    TODO: need to be revised once I implement chained cosmetic operators.
-    // TODO: This is still not working well for a[href], because there are
+
-    // many ways to compose a valid href to the same effective URL.
+*/
-    // One idea is to normalize all a[href] on the page, but for now I will
+var filterToDOMInterface = (function() {
-    // wait and see, as I prefer to refrain from tampering with the page
+    // Net filters: we need to lookup manually -- translating into a foolproof
-    // content if I can avoid it.
+    // CSS selector is just not possible.
-    var elems, iElem, elem;
+    var fromNetworkFilter = function(filter) {
-    if ( filter.lastIndexOf('##', 0) === 0 ) {
+        var out = [];
+        // https://github.com/chrisaljoudi/uBlock/issues/945
+        // Transform into a regular expression, this allows the user to edit and
+        // insert wildcard(s) into the proposed filter.
+        var reStr = '';
+        if ( filter.length > 1 && filter.charAt(0) === '/' && filter.slice(-1) === '/' ) {
+            reStr = filter.slice(1, -1);
+        }
+        else {
+            var rePrefix = '', reSuffix = '';
+            if ( filter.slice(0, 2) === '||' ) {
+                filter = filter.replace('||', '');
+            } else {
+                if ( filter.charAt(0) === '|' ) {
+                    rePrefix = '^';
+                    filter = filter.slice(1);
+                }
+            }
+            if ( filter.slice(-1) === '|' ) {
+                reSuffix = '$';
+                filter = filter.slice(0, -1);
+            }
+            reStr = rePrefix +
+                    filter.replace(/[.+?${}()|[\]\\]/g, '\\$&').replace(/[\*^]+/g, '.*') +
+                    reSuffix;
+        }
+        var reFilter = null;
         try {
-            elems = document.querySelectorAll(filter.slice(2));
+            reFilter = new RegExp(reStr);
         }
         catch (e) {
-            elems = [];
+            return out;
         }
-        iElem = elems.length;
+
+        // Lookup by tag names.
+        var src1stProps = netFilter1stSources;
+        var src2ndProps = netFilter2ndSources;
+        var srcProp, src;
+        var elems = document.querySelectorAll(Object.keys(src1stProps).join()),
+            iElem = elems.length,
+            elem;
         while ( iElem-- ) {
-            out.push({
+            elem = elems[iElem];
-                type: 'cosmetic',
+            srcProp = src1stProps[elem.localName];
-                elem: elems[iElem],
+            src = elem[srcProp];
-            });
+            if ( typeof src !== 'string' || src.length === 0 ) {
-        }
+                srcProp = src2ndProps[elem.localName];
-        return out;
+                src = elem[srcProp];
-    }
+            }
-
+            if ( src && reFilter.test(src) ) {
-    // Net filters: we need to lookup manually -- translating into a
+                out.push({
-    // foolproof CSS selector is just not possible
+                    type: 'network',
-
+                    elem: elem,
-    // https://github.com/chrisaljoudi/uBlock/issues/945
+                    src: srcProp,
-    // Transform into a regular expression, this allows the user to edit and
+                    opts: filterTypes[elem.localName],
-    // insert wildcard(s) into the proposed filter
+                });
-    var reStr = '';
-    if ( filter.length > 1 && filter.charAt(0) === '/' && filter.slice(-1) === '/' ) {
-        reStr = filter.slice(1, -1);
-    }
-    else {
-        var rePrefix = '', reSuffix = '';
-        if ( filter.slice(0, 2) === '||' ) {
-            filter = filter.replace('||', '');
-        } else {
-            if ( filter.charAt(0) === '|' ) {
-                rePrefix = '^';
-                filter = filter.slice(1);
             }
         }
-        if ( filter.slice(-1) === '|' ) {
+
-            reSuffix = '$';
+        // Find matching background image in current set of candidate elements.
-            filter = filter.slice(0, -1);
+        elems = candidateElements;
+        iElem = elems.length;
+        while ( iElem-- ) {
+            elem = elems[iElem];
+            if ( reFilter.test(backgroundImageURLFromElement(elem)) ) {
+                out.push({
+                    type: 'network',
+                    elem: elem,
+                    style: 'background-image',
+                    opts: 'image',
+                });
+            }
         }
-        reStr = rePrefix +
+
-                filter.replace(/[.+?${}()|[\]\\]/g, '\\$&').replace(/[\*^]+/g, '.*') +
-                reSuffix;
-    }
-    var reFilter = null;
-    try {
-        reFilter = new RegExp(reStr);
-    }
-    catch (e) {
         return out;
-    }
+    };
 
-    // Lookup by tag names.
+    // Cosmetic filters: these are straight CSS selectors.
-    var src1stProps = netFilter1stSources;
+    // TODO: This is still not working well for a[href], because there are many
-    var src2ndProps = netFilter2ndSources;
+    // ways to compose a valid href to the same effective URL. One idea is to
-    var srcProp, src;
+    // normalize all a[href] on the page, but for now I will wait and see, as I
-    elems = document.querySelectorAll(Object.keys(src1stProps).join());
+    // prefer to refrain from tampering with the page content if I can avoid it.
-    iElem = elems.length;
+    var fromCosmeticFilter = function(filter) {
-    while ( iElem-- ) {
+        var elems;
-        elem = elems[iElem];
+        try {
-        srcProp = src1stProps[elem.localName];
+            elems = document.querySelectorAll(filter);
-        src = elem[srcProp];
-        if ( typeof src !== 'string' || src.length === 0 ) {
-            srcProp = src2ndProps[elem.localName];
-            src = elem[srcProp];
         }
-        if ( src && reFilter.test(src) ) {
+        catch (e) {
-            out.push({
+            return fromProceduralCosmeticFilter(filter);
-                type: 'network',
-                elem: elem,
-                src: srcProp,
-                opts: filterTypes[elem.localName],
-            });
         }
-    }
+        var out = [],
+            iElem = elems.length;
+        while ( iElem-- ) {
+            out.push({ type: 'cosmetic', elem: elems[iElem]});
+        }
+        return out;
+    };
 
-    // Find matching background image in current set of candidate elements.
+    // https://github.com/gorhill/uBlock/issues/1772
-    elems = candidateElements;
+    // Handle procedural cosmetic filters.
-    iElem = elems.length;
+    var fromProceduralCosmeticFilter = function(filter) {
-    while ( iElem-- ) {
+        if ( filter.charCodeAt(filter.length - 1) === 0x29 /* ')' */ ) {
-        elem = elems[iElem];
+            var parts = reProceduralCosmeticFilter.exec(filter);
-        if ( reFilter.test(backgroundImageURLFromElement(elem)) ) {
+            if (
-            out.push({
+                parts !== null &&
-                type: 'network',
+                proceduralCosmeticFilterFunctions.hasOwnProperty(parts[2])
-                elem: elem,
+            ) {
-                style: 'background-image',
+                return proceduralCosmeticFilterFunctions[parts[2]](
-                opts: 'image',
+                    parts[1].trim(),
-            });
+                    parts[3].trim()
+                );
+            }
         }
-    }
+    };
 
-    return out;
+    var reProceduralCosmeticFilter = /^(.*?):(matches-css|has|style|xpath)\((.+?)\)$/;
-};
+
+    // Collection of handlers for procedural cosmetic filters.
+    var proceduralCosmeticFilterFunctions = {
+        'has': function(selector, arg) {
+            if ( selector === '' ) { return; }
+            var elems;
+            try {
+                elems = document.querySelectorAll(selector);
+                document.querySelector(arg);
+            } catch(ex) {
+                return;
+            }
+            var out = [];
+            for ( var i = 0, n = elems.length; i < n; i++ ) {
+                if ( document.querySelector(arg) ) {
+                    out.push({ type: 'cosmetic', elem: elems[i] });
+                }
+            }
+            return out;
+        },
+        'matches-css': function(selector, arg) {
+            if ( selector === '' ) { return; }
+            var elems;
+            try {
+                elems = document.querySelectorAll(selector);
+            } catch(ex) {
+                return;
+            }
+            var out = [], elem, style,
+                pos = arg.indexOf(':');
+            if ( pos === -1 ) { return; }
+            var prop = arg.slice(0, pos).trim(),
+                reText = arg.slice(pos + 1).trim();
+            if ( reText === '' ) { return; }
+            var re = reText !== '*' ?
+                new RegExp('^' + reText.replace(/[.+?${}()|[\]\\^]/g, '\\$&').replace(/\*+/g, '.*?') + '$') :
+                /./;
+            for ( var i = 0, n = elems.length; i < n; i++ ) {
+                elem = elems[i];
+                style = window.getComputedStyle(elem, null);
+                if ( re.test(style[prop]) ) {
+                    out.push({ type: 'cosmetic', elem: elem });
+                }
+            }
+            return out;
+        },
+        'style': function(selector, arg) {
+            if ( selector === '' || arg === '' ) { return; }
+            var elems;
+            try {
+                elems = document.querySelectorAll(selector);
+            } catch(ex) {
+                return;
+            }
+            var out = [];
+            for ( var i = 0, n = elems.length; i < n; i++ ) {
+                out.push({ type: 'cosmetic', elem: elems[i] });
+            }
+            lastAction = selector + ' { ' + arg + ' }';
+            return out;
+        },
+        'xpath': function(selector, arg) {
+            if ( selector !== '' ) { return []; }
+            var result;
+            try {
+                result = document.evaluate(
+                    arg,
+                    document,
+                    null,
+                    XPathResult.UNORDERED_NODE_SNAPSHOT_TYPE,
+                    null
+                );
+            } catch(ex) {
+                return;
+            }
+            if ( result === undefined ) { return []; }
+            var out = [], elem, i = result.snapshotLength;
+            while ( i-- ) {
+                elem = result.snapshotItem(i);
+                if ( elem.nodeType === 1 ) {
+                    out.push({ type: 'cosmetic', elem: elem });
+                }
+            }
+            return out;
+        }
+    };
+
+    var lastFilter,
+        lastResultset,
+        lastAction,
+        appliedStyleTag,
+        applied = false,
+        previewing = false;
+
+    var queryAll = function(filter) {
+        filter = filter.trim();
+        if ( filter === lastFilter ) {
+            return lastResultset;
+        }
+        unapply();
+        if ( filter === '' ) {
+            lastFilter = '';
+            lastResultset = [];
+        } else {
+            lastFilter = filter;
+            lastAction = undefined;
+            lastResultset = filter.lastIndexOf('##', 0) === 0 ?
+                fromCosmeticFilter(filter.slice(2)) :
+                fromNetworkFilter(filter);
+            if ( previewing ) {
+                apply(filter);
+            }
+        }
+        return lastResultset;
+    };
+
+    var applyHide = function() {
+        var htmlElem = document.documentElement,
+            items = lastResultset,
+            item, elem, style;
+        for ( var i = 0, n = items.length; i < n; i++ ) {
+            item = items[i];
+            elem = item.elem;
+            // https://github.com/gorhill/uBlock/issues/1629
+            if ( elem === pickerRoot ) {
+                continue;
+            }
+            style = elem.style;
+            if (
+                (elem !== htmlElem) &&
+                (item.type === 'cosmetic' || item.type === 'network' && item.src !== undefined)
+            ) {
+                item.display = style.getPropertyValue('display');
+                item.displayPriority = style.getPropertyPriority('display');
+                style.setProperty('display', 'none', 'important');
+            }
+            if ( item.type === 'network' && item.style === 'background-image' ) {
+                item.backgroundImage = style.getPropertyValue('background-image');
+                item.backgroundImagePriority = style.getPropertyPriority('background-image');
+                style.setProperty('background-image', 'none', 'important');
+            }
+        }
+    };
+
+    var unapplyHide = function() {
+        var items = lastResultset, item;
+        for ( var i = 0, n = items.length; i < n; i++ ) {
+            item = items[i];
+            if ( item.hasOwnProperty('display') ) {
+                item.elem.style.setProperty(
+                    'display',
+                    item.display,
+                    item.displayPriority
+                );
+                delete item.display;
+            }
+            if ( item.hasOwnProperty('backgroundImage') ) {
+                item.elem.style.setProperty(
+                    'background-image',
+                    item.backgroundImage,
+                    item.backgroundImagePriority
+                );
+                delete item.backgroundImage;
+            }
+        }
+    };
+
+    var unapplyStyle = function() {
+        if ( !appliedStyleTag || appliedStyleTag.parentNode === null ) {
+            return;
+        }
+        appliedStyleTag.parentNode.removeChild(appliedStyleTag);
+    };
+
+    var applyStyle = function() {
+        if ( !appliedStyleTag ) {
+            appliedStyleTag = document.createElement('style');
+            appliedStyleTag.setAttribute('type', 'text/css');
+        }
+        appliedStyleTag.textContent = lastAction;
+        if ( appliedStyleTag.parentNode === null ) {
+            document.head.appendChild(appliedStyleTag);
+        }
+    };
+
+    var apply = function() {
+        if ( applied ) {
+            unapply();
+        }
+        if ( lastResultset === undefined ) {
+            return;
+        }
+        if ( typeof lastAction === 'string' ) {
+            applyStyle();
+        } else {
+            applyHide();
+        }
+        applied = true;
+    };
+
+    var unapply = function() {
+        if ( !applied ) {
+            return;
+        }
+        if ( typeof lastAction === 'string' ) {
+            unapplyStyle();
+        } else {
+            unapplyHide();
+        }
+        applied = false;
+    };
+
+    var preview = function(filter) {
+        previewing = filter !== false;
+        if ( previewing ) {
+            if ( queryAll(filter) !== undefined ) {
+                apply();
+            }
+        } else {
+            unapply();
+        }
+        pickerBody.classList.toggle('preview', previewing);
+    };
+
+    return {
+        previewing: function() { return previewing; },
+        preview: preview,
+        set: queryAll
+    };
+})();
 
 // https://www.youtube.com/watch?v=nuUXJ6RfIik
 
@@ -810,8 +1004,8 @@ var elementsFromFilter = function(filter) {
 
 var userFilterFromCandidate = function() {
     var v = taCandidate.value;
-    var items = elementsFromFilter(v);
+    var items = filterToDOMInterface.set(v);
-    if ( items.length === 0 ) {
+    if ( !items || items.length === 0 ) {
         return false;
     }
 
@@ -850,13 +1044,18 @@ var userFilterFromCandidate = function() {
 /******************************************************************************/
 
 var onCandidateChanged = function() {
-    unpreview();
+    var elems = [],
-
+        items = filterToDOMInterface.set(taCandidate.value),
-    var elems = [];
+        valid = items !== undefined;
-    var items = elementsFromFilter(taCandidate.value);
+    if ( valid ) {
-    for ( var i = 0; i < items.length; i++ ) {
+        for ( var i = 0; i < items.length; i++ ) {
-        elems.push(items[i].elem);
+            elems.push(items[i].elem);
+        }
     }
+    pickerBody.querySelector('body section textarea + div').textContent = valid ?
+        items.length.toLocaleString() :
+        '0';
+    taCandidate.classList.toggle('invalidFilter', !valid);
     dialog.querySelector('#create').disabled = elems.length === 0;
     highlightElements(elems, true);
 };
@@ -885,18 +1084,23 @@ var candidateFromFilterChoice = function(filterChoice) {
     if ( filterChoice.modifier ) {
         return filter.replace(/:nth-of-type\(\d+\)/, '');
     }
-    
+
     // Return path: the target element, then all siblings prepended
-    var selector = [];
+    var selector = '', joiner = '';
     for ( ; slot < filters.length; slot++ ) {
         filter = filters[slot];
-        selector.unshift(filter.replace(/^##/, ''));
+        selector = filter.slice(2) + joiner + selector;
         // Stop at any element with an id: these are unique in a web page
-        if ( filter.slice(0, 3) === '###' ) {
+        if ( filter.lastIndexOf('###', 0) === 0 ) {
             break;
         }
+        // Stop if current selector matches only one element on the page
+        if ( document.querySelectorAll(selector).length === 1 ) {
+            break;
+        }
+        joiner = ' > ';
     }
-    return '##' + selector.join(' > ');
+    return '##' + selector;
 };
 
 /******************************************************************************/
@@ -926,8 +1130,7 @@ var onDialogClicked = function(ev) {
     else if ( ev.target.id === 'create' ) {
         // We have to exit from preview mode: this guarantees matching elements
         // will be found for the candidate filter.
-        unpreview();
+        filterToDOMInterface.preview(false);
-
         var filter = userFilterFromCandidate();
         if ( filter ) {
             var d = new Date();
@@ -939,7 +1142,7 @@ var onDialogClicked = function(ev) {
                     pageDomain: window.location.hostname
                 }
             );
-            filterElements(taCandidate.value);
+            filterToDOMInterface.preview(taCandidate.value);
             stopPicker();
         }
     }
@@ -949,15 +1152,15 @@ var onDialogClicked = function(ev) {
     }
 
     else if ( ev.target.id === 'quit' ) {
-        unpreview();
+        filterToDOMInterface.preview(false);
         stopPicker();
     }
 
     else if ( ev.target.id === 'preview' ) {
-        if ( pickerBody.classList.contains('preview') ) {
+        if ( filterToDOMInterface.previewing() ) {
-            unpreview();
+            filterToDOMInterface.preview(false);
         } else {
-            preview(taCandidate.value);
+            filterToDOMInterface.preview(taCandidate.value);
         }
         highlightElements(targetElements, true);
     }
@@ -1068,9 +1271,13 @@ var onSvgHovered = (function() {
 
 var onSvgClicked = function(ev) {
     // https://github.com/chrisaljoudi/uBlock/issues/810#issuecomment-74600694
-    // Unpause picker if user click outside dialog
+    // Unpause picker if:
+    // - click outside dialog AND
+    // - not in preview mode
     if ( pickerBody.classList.contains('paused') ) {
-        unpausePicker();
+        if ( filterToDOMInterface.previewing() === false ) {
+            unpausePicker();
+        }
         return;
     }
     if ( filtersFrom(ev.clientX, ev.clientY) === 0 ) {
@@ -1116,7 +1323,7 @@ var pausePicker = function() {
 /******************************************************************************/
 
 var unpausePicker = function() {
-    unpreview();
+    filterToDOMInterface.preview(false);
     pickerBody.classList.remove('paused');
     svgListening(true);
 };
@@ -1127,10 +1334,11 @@ var unpausePicker = function() {
 // in use: to ensure this, release all local references.
 
 var stopPicker = function() {
+    vAPI.shutdown.remove(stopPicker);
+
     targetElements = [];
     candidateElements = [];
     bestCandidateFilter = null;
-    previewedElements = [];
 
     if ( pickerRoot === null ) {
         return;
@@ -1262,24 +1470,25 @@ var startPicker = function(details) {
 pickerRoot = document.createElement('iframe');
 pickerRoot.id = vAPI.sessionId;
 pickerRoot.style.cssText = [
-    'display: block',
-    'visibility: visible',
-    'opacity: 1',
-    'position: fixed',
-    'top: 0',
-    'left: 0',
-    'width: 100%',
-    'height: 100%',
     'background: transparent',
-    'margin: 0',
-    'padding: 0',
     'border: 0',
     'border-radius: 0',
     'box-shadow: none',
+    'display: block',
+    'height: 100%',
+    'left: 0',
+    'margin: 0',
+    'max-height: none',
+    'opacity: 1',
     'outline: 0',
+    'padding: 0',
+    'position: fixed',
+    'top: 0',
+    'visibility: visible',
+    'width: 100%',
     'z-index: 2147483647',
     ''
-].join('!important;');
+].join(' !important;');
 
 // https://github.com/gorhill/uBlock/issues/1529
 // In addition to inline styles, harden the element picker styles by using
@@ -1307,8 +1516,6 @@ pickerRoot.onload = function() {
 
 document.documentElement.appendChild(pickerRoot);
 
-/******************************************************************************/
-
 // https://www.youtube.com/watch?v=sociXdKnyr8
 
 /******************************************************************************/