Fix Cursor Placement in Checkbox Lists by pointing to the correct node (cross-browser improvement) #413
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR addresses a cursor placement issue in checkbox lists
Previously,
setCollapse()
was called on the<ol>
node. This works correctly in Chrome, Safari, and all mobile WebViews, but the online example does not behave correctly in Firefox desktop and the cursor is placed incorrectly after the checkbox.This change ensures that
setCollapse()
points to the<li>
node containing the checkbox, which aligns with the correct HTML semantics and guarantees consistent behavior across all browsers, including Firefox desktop.Notes
No changes affect Chrome, Safari, or mobile WebViews; they continue to function as before.
This fix is only required for the Firefox desktop online example due to its stricter handling of contenteditable and non-editable elements.
Improves cross-browser consistency for checkbox lists in the online example.
Does not affect other parts of the editor where setCollapse is used.
Changes