This patch adds multiple CSS selectors to sIFR. This results in less sIFR rules added to HTML output and a slightly better browser performance. Currently, only one CSS selector can be defined per rule.

sIFR additionally can support identical rule names. They're not used elsewhere, so it's no big thing if two rules have the same name. I've been messed up after renaming one of my rules. [$r->name to $r->rid]

Additionally added sIFR 2.0.2 CSS adblock snippet.

This patch is tested with 4.7.3.

CommentFileSizeAuthor
#2 sifr.multiple-selectors2.patch3.37 KBsun
sifr.multiple-selectors.patch3.31 KBsun

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Jeff, is it possible to commit this one?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Reviewed & tested by the community » Patch (to be ported)
StatusFileSize
new sifr.multiple-selectors2.patch3.37 KB

Comitted to 4.7.x. Needs to be ported to 5.x.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Patch (to be ported) » Fixed

alright, did it by myself.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)