-
Notifications
You must be signed in to change notification settings - Fork 64
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 anchor links handlers #245
Conversation
* Add three JS function to main.js, which given a DOM content block, traverse its header elements, and given that they include an id attribute, an anchor link element will be added. * As the project already uses Font Awesome, this link shows the fa-link icon. * Add the necssary CSS code to get this added element hidden by default, and show it on hover.
@@ -12,6 +13,34 @@ function activeToggle() { | |||
}); | |||
} | |||
|
|||
var anchorForId = function (id) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It could be great if we could move these functions to Scala.js code, as it's a short script! Maybe for a future PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's release a new patch version with this feature. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bumped version on bf5e574
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks!
This closes #74.