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

Fix completer tests after transition to pointer events #15237

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

krassowski
Copy link
Member

References

Follow-up after #14534

Code changes

  • switches simulated events in tests from mousedown to pointerdown
  • adds a polyfill for pointerdown to @jupyterlab/testing

User-facing changes

None

Backwards-incompatible changes

None

Verified

This commit was signed with the committer’s verified signature.
isaacs isaacs
@krassowski krassowski added this to the 3.6.x milestone Oct 8, 2023
@jupyterlab-probot
Copy link

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

Copy link
Member

@fcollonval fcollonval left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @krassowski

@fcollonval fcollonval merged commit 206368a into jupyterlab:main Oct 9, 2023
@fcollonval
Copy link
Member

@meeseeksdev please backport to 4.0.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Oct 9, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
… pointer events
@fcollonval
Copy link
Member

fcollonval commented Oct 9, 2023

@krassowski you set the target to 3.6.x is it to backport the polyfill only as the mentioned PR has not been backported to 3.6.x?

@krassowski
Copy link
Member Author

krassowski commented Oct 9, 2023

I set it to 3.6.x because #14534 was a bug fix to an issue in 3.6.x (but I did not backport it yet so indeed for now there is no need to backport this one either).

Edit: I will clean up backports in the evening.

@fcollonval
Copy link
Member

I added the tag Still needs manual backport to not forget about this.

krassowski added a commit that referenced this pull request Oct 9, 2023

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
…vents (#15243)

Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
krassowski added a commit to meeseeksmachine/jupyterlab that referenced this pull request Oct 9, 2023

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
… pointer events

Cherry-picked from commit 206368a.
krassowski added a commit to meeseeksmachine/jupyterlab that referenced this pull request Oct 9, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
… pointer events

Cherry-picked from commit 206368a.
krassowski added a commit that referenced this pull request Oct 18, 2023
… changing notebook tabs) (#15244)

* Backport PR #14534: Hide completer when changing notebook tabs

* Backport PR #15237: Fix completer tests after transition to pointer events

Cherry-picked from commit 206368a.

---------

Co-authored-by: Nishikant Parmar <nishikantparmar123@gmail.com>
Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants