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

Backport PR #15443 on branch 4.0.x (Fix search coming back in notebook and editor) #15562

Merged

Conversation

krassowski
Copy link
Member

Backport PR #15443 on branch 4.0.x (Fix search coming back in notebook and editor)

…editor

* a test case to illustrate issue jupyterlab#14871

* cleanup unused code

* rename outline into highlight as it sounds more proper

* fix typo in variable name _unrenderedByHighligh → _unrenderedByHighlight

* Fix search coming back in notebook and editor

* Add missing return

* Rename snapshots and limit screenshot to notebook area

Remove no-op await locator line

* Update Playwright Snapshots

* Improve async test implementation to avoid early closing

* Update Playwright Snapshots

* Remove visual snapshots, use locator counts instead

and revert spurious snapshot updates

---------

Co-authored-by: Thierry Parmentelat <thierry.parmentelat@inria.fr>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 746ce2d)
@krassowski krassowski added the bug label Dec 26, 2023
@krassowski krassowski added this to the 4.0.x milestone Dec 26, 2023
Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@krassowski
Copy link
Member Author

usage2 failures are unrelated and tracked in #15570, visual tests failures are unrelated (a fix needs backporting).

@krassowski krassowski merged commit dd68b9c into jupyterlab:4.0.x Dec 27, 2023
74 of 78 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant