-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Attention jumps to last known search result location when a cell operation is executed #15986
Comments
Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗 |
There has been a lot of work on improving search since JupyterLab 4.0.9 was released. There have been also security patches released since then. Can you please test with the latest version available (4.1.4 as of time of writing)? |
I just upgraded:
==> So now running on lab 4.1.4 And the issue still remains. |
related pr: 14837 |
For reference: jupyterlab/packages/notebook/src/searchprovider.ts Lines 560 to 591 in ee99e3c
This is indeed intended, the search should be updated because if a new cell is added it may contain matches; if a cell is removed matches may be removed; if cell is moved the index of current match may change. The bug is that it should not lead to scrolling to the current match. |
Additionally @krassowski, it's more than just the scrolling (although that is the most visible side effect): even when the search dialog is closed, it leads to the scrolling (and based on your comment I assume to the execution of the search code). |
Did you see this in the latest release? I would have thought that this was fixed by #15443 which was included in 4.0.10 so I am not surprised that you saw it in 4.0.9. |
no, i can see it in 4.1.15 |
You're right. I can't reproduce that bit anymore 🥳 |
😂 @svaningelgem Are you currently unable to reproduce in the latest release, but it can be reproduced in version 4.1.14? |
wow, the internet is not suitable for nuanced language 🤣
That this I can't reproduce anymore. Adding/deleting a cell still leads to scrolling. I hope I made the point clearer now 🤔 ? |
sorry , I didn't carefully read @krassowski's response in context |
Description
This problem was already once reported in #9437. However, it seems to still be present in the current versions?
Reproduce
This is highly annoying.
So I tried to close that dialog (cross). (the search results are NOT highlighted anymore - as expected)
Expected behavior
What I'd expect to see is when I search:
Context
Troubleshoot Output
Command Line Output
Browser Output
The text was updated successfully, but these errors were encountered: