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

bug: Selecting empty cells causes error in prosemirror #123

Closed
mkarbo opened this issue Mar 14, 2023 · 8 comments · Fixed by #140
Closed

bug: Selecting empty cells causes error in prosemirror #123

mkarbo opened this issue Mar 14, 2023 · 8 comments · Fixed by #140

Comments

@mkarbo
Copy link

mkarbo commented Mar 14, 2023

After initializing the editor, before creating any blocks, selecting the empty placeholder block causes an error in prosemirror
image

@YousefED
Copy link
Collaborator

This is likely to be a dependency installation issue. Could you share a reproducible project?

@mkarbo
Copy link
Author

mkarbo commented Mar 15, 2023

https://github.com/mkarbo/BlockNote-bug-123 reproduces it on a blank next project with latest alpha.

My original test was on 0.4.3

@YousefED
Copy link
Collaborator

I don't see the error using this project. Could you try clearing node_modules?

@mkarbo
Copy link
Author

mkarbo commented Mar 17, 2023

Just to be clear - the error doesn't automatically pop up for me on first render. I can provoke it by trying to select empty row, and sometimes it pops up when e.g., switching tabs.

I replicated this from a clean install, nothing to do with node modules I believe

@YousefED
Copy link
Collaborator

Thanks. Could you try 0.4.6-alpha.4 for a possible workaround?

@mkarbo
Copy link
Author

mkarbo commented Mar 17, 2023

Sure, will do

@mkarbo
Copy link
Author

mkarbo commented Mar 17, 2023

No longer able to provoke the issue manually! 👏

@YousefED
Copy link
Collaborator

nice!

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 a pull request may close this issue.

2 participants