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

Docs: Refactor link generation to allow more natural mouse interaction. #23071

Merged
merged 3 commits into from
Dec 21, 2022

Conversation

gero3
Copy link
Contributor

@gero3 gero3 commented Dec 23, 2021

Related issue: Fixed #13900

Description

This PR makes it possible to open the documentation in another tab by using modifier keys like shift, ctrl, alt, and by middle mouse button. This works because only the first mouse button and no modifier keys update only the hash fragment of the URL. The others use the standard behavior of the hyperlink element.
I've also taken special notice to not break the behavior of accessing the documentation with the file protocol

Here is the new documentation

Copy link
Collaborator

@Mugen87 Mugen87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tested the change with different doc pages and everything seems to work as expected.

@Mugen87 Mugen87 changed the title Modifier keys work with link in documentation Docs: Refactor link generation to allow more natural mouse interaction. Dec 28, 2021
@gero3
Copy link
Contributor Author

gero3 commented Jan 4, 2022

Is there anything I need to change on this PR??

@felixniemeyer
Copy link

I come from this issue that was marked as duplicate of this one, which is linked to this pull request.
I'm puzzled why this not getting merged.

@mrdoob mrdoob added this to the r148 milestone Dec 21, 2022
@mrdoob mrdoob merged commit 9d72c68 into mrdoob:dev Dec 21, 2022
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.

Links on three.js documentation website prevent ctrl/cmd+click or right click opening
4 participants