GeraldWodni / forth-standard.org

Official website of the Forth 2012 Standard, and the Forth200x committee.
9 stars 3 forks source link

Fix scrolling to a local fragment, tune #37

Closed ruv closed 2 years ago

ruv commented 2 years ago

Due to a mistake in a LESS code fragment, a generated CSS selector is: a[name] :target (i.e. a descendant of a[name] that is a target), but we need a[name]:target (i.e. a[name] that is a target).

Also, don't apply dotted bottom border to a target element when it is a div or section element, since usually they envelope both the title and body of a fragment. But we want to decorate only a title (if any).

GeraldWodni commented 2 years ago

Right, I also missed the wrong scoping. Less uses the & operator for that, so I changed it directly instead of merging this pull request in 16b56c00c970ee431f859d6d5c3530f7a409a327.

I am still curious what the dotted target border is intended for, can you please give an example link where this is visible? Thanks!

ruv commented 2 years ago

Examples of links that show a title that is decorated with a dotted underline:

https://forth-standard.org/standard/usage#usage:compile https://forth-standard.org/standard/usage#subsubsection.3.4.3.3

These both links refer to the same subsection "3.4.3.3 Compilation semantics"

Links to comments refer a surrounding (containing) div element, and there is no sense to underline the bottom border of this div. An example: https://forth-standard.org/standard/usage#contribution-166