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

Examples: scroll to currently selected example in the sidebar #28837

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

aardgoose
Copy link
Contributor

When an example is selected in the sidebar, and the page is refreshed, the sidebar reverts to the default positioning,

Scroll the sidebar so that the selected example name and thumbnail are visible.

@Mugen87 Mugen87 added this to the r167 milestone Jul 9, 2024
@mrdoob mrdoob merged commit 4ce3b36 into mrdoob:dev Jul 9, 2024
11 checks passed
John-Simth added a commit to John-Simth/three.js that referenced this pull request Jul 10, 2024
mrdoob pushed a commit that referenced this pull request Jul 10, 2024
Co-authored-by: aardgoose <[email protected]>
@aardgoose aardgoose deleted the scroll branch July 17, 2024 08:07
@Mugen87
Copy link
Collaborator

Mugen87 commented Jul 26, 2024

It seems this PR broke the menu bar on mobile devices. This can be reproduced with Chrome dev tools (device mode).

Prod: https://threejs.org/examples/index.html#webgl_animation_keyframes

image

One commit before merge: https://rawcdn.githack.com/mrdoob/three.js/ad94b23dbfe9ad8242154e548bbf9af9746820e8/examples/index.html#webgl_animation_keyframes

image

@Mugen87
Copy link
Collaborator

Mugen87 commented Jul 27, 2024

I have reverted the change in the gh-pages branch for now: ca42d00

Mugen87 added a commit that referenced this pull request Aug 1, 2024
Mugen87 added a commit that referenced this pull request Aug 1, 2024
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