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

Search "in selected" sometimes reverts to edit mode preventing cell selection #14308

Closed
krassowski opened this issue Apr 1, 2023 · 2 comments · Fixed by #14387
Closed

Search "in selected" sometimes reverts to edit mode preventing cell selection #14308

krassowski opened this issue Apr 1, 2023 · 2 comments · Fixed by #14387
Labels
bug tag:Release Blocker A must-have bug for the milestone to which it is tagged tag:Search and Replace
Milestone

Comments

@krassowski
Copy link
Member

krassowski commented Apr 1, 2023

Description

When searching "in selection" (#14090), switching to command mode and attempting to select multiple cells the mode switches back to edit mode putting cursor back in the cell and preventing selecting more cells.

This appears somewhat similar to seemingly unrelated issue #14271 (could be unrelated or could have to do with recent cursors work)

Reproduce

switches-to-edit-mode

Expected behavior

When Shift + Up/Shift + Down is pressed the cell selection extends and the command mode remains unchanged.

Context

  • JupyterLab version: 4.0.0b
@krassowski krassowski added bug tag:Search and Replace status:Needs Triage Applied to new issues that need triage labels Apr 1, 2023
@krassowski krassowski added this to the 4.0.0 milestone Apr 1, 2023
@JasonWeill JasonWeill removed the status:Needs Triage Applied to new issues that need triage label Apr 4, 2023
@krassowski
Copy link
Member Author

krassowski commented Apr 15, 2023

This looks hard, and originates from CodeMirror focusing the editor back when decorations get updated. Question upstream: https://discuss.codemirror.net/t/how-to-add-decoration-without-focusing-the-editor-on-dispatch/6283

It is still not clear to me why CodeMirror only focuses editor back on some but not all updates to decorations.

@fcollonval fcollonval added the tag:Release Blocker A must-have bug for the milestone to which it is tagged label May 2, 2023
@fcollonval
Copy link
Member

I tagged this as release blocker as the associated PR changes the API; just to be sure we merge it before the final release.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug tag:Release Blocker A must-have bug for the milestone to which it is tagged tag:Search and Replace
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants