Feature/anchor all elements

[hoetmaaiers]

Update to match all anchor elements with :target talked about in #280 .

[choldgraf]

very cool, I didn't know about this target selector, is it consistent across browsers?

[hoetmaaiers]

Supprisingly yes. I also learned about it thanks to @jorisvandenbossche 's research.

https://developer.mozilla.org/en-US/docs/Web/CSS/:target

[jorisvandenbossche]

Cool!

@hoetmaaiers it seems you have a conflict (because the actual dt change is not in master, so the diff should show that this is removed or updated.

The change from 80 to 60px offset is intentional? (it looks fine on the demo docs for this PR)

[hoetmaaiers]

The merge conflicts are resolved now.

The choice for 60px is indeed on purpose. This leave some extra white space at the top without the anchor element being smacked to the edge.

[hoetmaaiers]

Grmbl, the merge resolution made the 'lint' check fail. @choldgraf or @jorisvandenbossche I always struggle with this check. Is it a problem with indentation or alike?

[jorisvandenbossche]

@hoetmaaiers the issue was that yarn build:production was still producing a change, and particularly the removal of the old built css/js files.
(after doing a yarn build, you need to do git add pydata_sphinx_theme/static/ to ensure the deleted files are also tracked as deleted by git)

[choldgraf]

this LGTM! and the tests are now passing with @jorisvandenbossche 's fix 🚀