Fix broken styling of selectors with pseudo elements

Related issue:
- https://github.com/uBlockOrigin/uBlock-issues/issues/382

Additionally, fix minor parsing issues with AdGuard's
cosmetic filters.
This commit is contained in:
Raymond Hill 2020-09-14 14:58:25 -04:00
parent 601ec67ab6
commit 328d6a09f4
No known key found for this signature in database
GPG key ID: 25E1490B761470C2
2 changed files with 7 additions and 6 deletions

View file

@ -777,11 +777,6 @@ vAPI.injectScriptlet = function(doc, text) {
for ( const raw of selectors ) {
if ( this.selectors.has(raw) ) { continue; }
const o = JSON.parse(raw);
if ( o.pseudo !== undefined ) {
this.domFilterer.addCSSRule(o.selector, vAPI.hideStyle);
mustCommit = true;
continue;
}
// CSS selector-based styles.
if (
o.action !== undefined &&
@ -792,6 +787,11 @@ vAPI.injectScriptlet = function(doc, text) {
mustCommit = true;
continue;
}
if ( o.pseudo !== undefined ) {
this.domFilterer.addCSSRule(o.selector, vAPI.hideStyle);
mustCommit = true;
continue;
}
let style, styleToken;
if ( o.action === undefined ) {
style = vAPI.hideStyle;

View file

@ -1215,6 +1215,7 @@ Parser.prototype.SelectorCompiler = class {
if ( hasBits(this.parser.flavorBits, BITFlavorExtStyle) ) {
raw = this.translateAdguardCSSInjectionFilter(raw);
if ( raw === '' ) { return false; }
this.parser.flavorBits &= ~BITFlavorExtStyle;
out.raw = raw;
}
@ -1264,7 +1265,7 @@ Parser.prototype.SelectorCompiler = class {
}
translateAdguardCSSInjectionFilter(suffix) {
const matches = /^([^{]+)\{([^}]+)\}\s*$/.exec(suffix);
const matches = /^(.*)\s*\{([^}]+)\}\s*$/.exec(suffix);
if ( matches === null ) { return ''; }
const selector = matches[1].trim();
const style = matches[2].trim();