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

add handling for a DOM selection inconsistency #1

Open
wants to merge 1 commit into
base: deepnote
Choose a base branch
from

Conversation

jankuca
Copy link
Collaborator

@jankuca jankuca commented Jan 25, 2024

No description provided.

Copy link

@the21st the21st left a comment

Choose a reason for hiding this comment

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

@jankuca btw have you checked whether the latest version of slate has this issue fixed?

@vojtatranta
Copy link

This case, which Simon noticed on my previous version of PR seems to be OK now. https://github.com/deepnote/deepnote/pull/12877#pullrequestreview-1840968773

It seems that jumping doesn't occur even if I add breakpoinst where I could reproduce the error quite reliably.
So fingers crossed.

Copy link

@vojtatranta vojtatranta left a comment

Choose a reason for hiding this comment

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

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

Successfully merging this pull request may close these issues.

3 participants