WYSIWYG: Table cell format handling clears more styles than expected #4669

Closed
opened 2026-02-05 09:10:13 +03:00 by OVERLORD · 0 comments
Owner

Originally created by @ssddanbrown on GitHub (Apr 21, 2024).

This is due to cell range handling added to help clear formatting: c290d01adb/resources/js/wysiwyg/fixes.js (L92-L103)

The FormatRemove event is much wider than just format clearing calls, This is also fired on un-toggle of formats (bold, italic etc..) and format changes (alignment changes etc...). This needs to be better scoped to only action upon format remove button presses (Might need custom button action).

Originally created by @ssddanbrown on GitHub (Apr 21, 2024). This is due to cell range handling added to help clear formatting: https://github.com/BookStackApp/BookStack/blob/c290d01adbbdeeb68c4b0d791fe919a02432d457/resources/js/wysiwyg/fixes.js#L92-L103 The `FormatRemove` event is much wider than just format clearing calls, This is also fired on un-toggle of formats (bold, italic etc..) and format changes (alignment changes etc...). This needs to be better scoped to only action upon format remove button presses (Might need custom button action).
OVERLORD added the 🐛 Bug📝 WYSIWYG Editor💻 Front-End labels 2026-02-05 09:10:13 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#4669