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.

Sign up for GitHub

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

Jump to bottom

Shorten anchor links #315

Merged
merged 3 commits into from
Oct 5, 2020
Merged

Shorten anchor links #315

merged 3 commits into from
Oct 5, 2020

Conversation

SimonCropp
Copy link
Owner

@SimonCropp SimonCropp commented Oct 5, 2020 •
edited
Loading

snippet anchors will now be hashed to improve the signal to noise ratio

image

@SimonCropp SimonCropp added this to the 21.4.0 milestone Oct 5, 2020
@SimonCropp
Copy link
Owner Author

@claremacrae this might interest you

@SimonCropp SimonCropp merged commit d1f3e18 into master Oct 5, 2020
@SimonCropp SimonCropp deleted the shortenAnchorLinks branch October 5, 2020 21:12
@khalidabuhakmeh
Copy link
Contributor

word to the wise, every element that has an id gets a DOM accessor on the window object.

This could interfere with client-side JavaScript that folks are writing around the markdown files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Feature
Projects
None yet
Milestone
21.4.0
Development

Successfully merging this pull request may close these issues.
2 participants
@SimonCropp @khalidabuhakmeh