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

Ignore mutations happens in nested Quill instance #3223

Merged
merged 1 commit into from
Nov 12, 2020

Conversation

luin
Copy link
Member

@luin luin commented Nov 11, 2020

Closes #1847

Previously, mutations generated inside nested Quill instances will be also handled by the parent, which is not expected.

This PR checks whether the scrolls of the blots are the current one and will ignore them if that isn't the case.

@luin luin requested a review from jhchen November 11, 2020 07:13
@jhchen jhchen merged commit 8ce3ee3 into slab:develop Nov 12, 2020
@luin luin deleted the ignore-nested-quill-mutations branch November 12, 2020 05:23
DokaRus pushed a commit to DokaRus/devextreme-quill that referenced this pull request Mar 25, 2021
Ignore mutations happens in nested Quill instance

(cherry picked from commit 8ce3ee3)
DokaRus added a commit to DevExpress/devextreme-quill that referenced this pull request Mar 26, 2021
* Merge pull request slab#3223 from luin/ignore-nested-quill-mutations

Ignore mutations happens in nested Quill instance

(cherry picked from commit 8ce3ee3)

* Merge pull request slab#3272 from luin/firefox-bullet

Header/paragraph doesn't reset list numbering in Firefox

(cherry picked from commit eeb976e)

* allow list completion on lists and headers

(cherry picked from commit a6911aa)

* fix arrow keying past list item

- in chrome it take two left arrows to go to prev list item
- in firefox ctrl+left does not work

(cherry picked from commit 9d16aa5)

* fix breaking list items

Without this a string with no space on a line will cause the break to
occur after the bullet but with it a string with spaces may not break at
the space. The latter is much more common and also the choice Paper
makes

(cherry picked from commit 232b6f4)

* Merge branch 'fix-cut-format'

* Merge pull request slab#3293 from quilljs/firefox-checkbox

Fix checkbox not checkable on Firefox

(cherry picked from commit 1f0530a)

Co-authored-by: Jason Chen <jhchen7@gmail.com>
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.

Nested Quill Editors and MutationObserver
2 participants