Skip to content

Patch handling of singleClickNavigation #7627

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

Closed
wants to merge 1 commit into from

Conversation

jtpio
Copy link
Member

@jtpio jtpio commented Apr 9, 2025

Follow-up to #7624

Fixes #7538

@jtpio jtpio added the bug label Apr 9, 2025
@jtpio jtpio added this to the 7.4.x milestone Apr 9, 2025
Copy link
Contributor

github-actions bot commented Apr 9, 2025

Binder 👈 Launch a Binder on branch jtpio/notebook/fix-single-click-navigation

@jtpio jtpio force-pushed the fix-single-click-navigation branch from 79ff9a9 to 375ffca Compare April 9, 2025 17:50
@jtpio
Copy link
Member Author

jtpio commented Apr 9, 2025

@afshin this seems to be fixing the issue, when using Firefox (138.0b5):

On main / 7.4.0

notebook-singleclick-navigation-issue.mp4

With this PR

notebook-singleclick-navigation.mp4

But it would be useful if some other people could double check too.

@RRosio
Copy link
Collaborator

RRosio commented Apr 15, 2025

I tried these changes locally and they resolve the issue for me as well.

@jtpio
Copy link
Member Author

jtpio commented Apr 15, 2025

Nice, thanks @RRosio for checking!

Do you want to submit the fix upstream in JupyterLab? Unless @afshin was planning to do it?

@afshin
Copy link
Member

afshin commented Apr 15, 2025

I wasn't planning to, so if someone else is happy to, from my point of view testing is the actual work here, I was just guessing 🤓

@jtpio
Copy link
Member Author

jtpio commented Apr 16, 2025

ok I opened jupyterlab/jupyterlab#17475 then, so we can have the fix available in JupyterLab 4.4.1 and update Notebook with the new version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'navigate files and folders with single click' only effective in jupyter lab
3 participants