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

[MSKINS-167] Add deep anchors to headers #16

Merged
merged 1 commit into from
Apr 19, 2020
Merged

[MSKINS-167] Add deep anchors to headers #16

merged 1 commit into from
Apr 19, 2020

Conversation

dialaya
Copy link
Contributor

@dialaya dialaya commented Apr 14, 2020

Hi,
This pull add a use of AnchorJS utility to generate deep anchors to sections header

The Permalink icon is displayed on Mouse-Over.
This helps on sharing links.

This closes #167

Add anchors to sections header using AnchorJS utility.

The Permalink icon is displayed on Mouse-Over.
This helps on sharing links.

This closes #167
@hboutemy hboutemy merged commit 3ad22d3 into apache:master Apr 19, 2020
@hboutemy
Copy link
Member

thank you @dialaya , that's really an awesome improvement

@hboutemy hboutemy self-requested a review April 19, 2020 17:51
@michael-o
Copy link
Member

This should have been distributed with the site resources and not loaded from the Internet. Just like we do with Bootstrap and other resources.

@hboutemy
Copy link
Member

@dialaya can you have a look please to host the lib in the skin instead of using CDN, please?

@hboutemy
Copy link
Member

@dialaya FYI, I just upgraded the skin used by Maven site itself, the result is great https://maven.apache.org/ :)

@dialaya
Copy link
Contributor Author

dialaya commented Apr 19, 2020 via email

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