I think this regressed in v2.2.23, around commit 6b27300, where selector normalization changed from:
to:
parsed = unescape(selectors).
That seems to have fixed #152 / #153 by unescaping selector text that had already been percent-encoded upstream.
The problem here is that the fix looks too broad and now unescapes valid escaped text inside quoted attribute substring selectors, e.g.
against HTML like:
<a href="/dperini/nwsapi/pulls?q=is%3Apr+is%3Aclosed">Closed</a>
That selector used to match, but after the unescape(selectors) change it effectively becomes [href*="is:pr"], which won't match.
I think this regressed in
v2.2.23, around commit 6b27300, where selector normalization changed from:to:
That seems to have fixed #152 / #153 by unescaping selector text that had already been percent-encoded upstream.
The problem here is that the fix looks too broad and now unescapes valid escaped text inside quoted attribute substring selectors, e.g.
against HTML like:
That selector used to match, but after the
unescape(selectors)change it effectively becomes[href*="is:pr"], which won't match.