search

readthedocs / common

Shared bits around multiple repositories
MIT License
22 stars 38 forks source link

CF worker: remove `script[id=READTHEDOCS_DATA] + script` #214

Closed humitos closed 5 months ago

humitos commented 5 months ago

We used to removed this previously, but the CSS selector was broken, so we stopped doing it. Today, I found the correct selector to remove it.

These are the two script this commit removes: https://github.com/readthedocs/readthedocs-sphinx-ext/blob/main/readthedocs_ext/_templates/readthedocs-insert.html.tmpl#L23-L31

humitos commented 5 months ago

Hrm... it seems that Cloudflare does not support this selector :/

 Parser error: Unsupported combinator `+` in selector.
ericholscher commented 5 months ago

Yea, CSS selectors are dark magic :/