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

Fixing errors for CM6 code input. #331

Closed
wants to merge 2 commits into from
Closed

Conversation

atodoron
Copy link

Fixing error where once a CM6 field has been opened it can't be opened again without a page refresh, as documented in #296.

Your Name added 2 commits November 10, 2024 22:57
Removing {signal} reference to listener which seems to reference an
already removed object.
@fregante
Copy link
Owner

Thank you for looking into this! So this fixed it?

I'll have to check what this piece of code was doing first.

@atodoron
Copy link
Author

I think so.

I've build and run it in Firefox and Chromium and I can edit the same code field multiple times in Jupyter notebooks.

I'm not great with JS so I have no idea what the side effects of what I've done are, but nothing catastrophic seems to be happening.

Copy link
Owner

@fregante fregante left a comment

Choose a reason for hiding this comment

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

Tested, this does not fix it.

table

@fregante fregante closed this Nov 11, 2024
@fregante fregante reopened this Nov 11, 2024
@fregante fregante closed this Nov 16, 2024
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.

2 participants