Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add heading hover-id-links #115

Merged
merged 2 commits into from
Nov 1, 2020
Merged

Add heading hover-id-links #115

merged 2 commits into from
Nov 1, 2020

Conversation

Kissaki
Copy link
Member

@Kissaki Kissaki commented Oct 31, 2020

For headings with an id attribute add anchors that show on hover and are clickable.
They link to the #id hash so clicking them will jump to that headline and update the address bar,
and copying their link will copy a link to that page and heading.

screenshot of hover anchor

The anchor will show on hover anywhere on the heading (not just to the left of it where the anchor link is).

This also does not break our 1.3.0 release notes blog post which had anchors at the end of them.

screenshot of double-anchors in 1.3.0 blog post

The unicode link character could be replaced with an icon. A blue color to match our hyperlinks could be good. We could also change that later though.

This change is especially useful for longer blog posts like release announcements, and will be useful for longer content pages, especially documentation.

Implements #43.

For headings with an id attribute add anchors that show on hover and are clickable.
They link to the #id hash so clicking them will jump to that headline and update the address bar,
and copying their link will copy a link to that page and heading.
@Kissaki Kissaki changed the title Add heading-hover id-links Add heading hover-id-links Oct 31, 2020
This was previously implemented in ad5f7e1.

Now we have a better mechanism. Only with JavaScript but for all headings with id attributes.
@Kissaki Kissaki merged commit e2790e2 into master Nov 1, 2020
@Kissaki Kissaki deleted the f/heading-links branch November 1, 2020 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants