Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
The CKEditor configuration UI allows tabs to be used in toolbar group names, which is fine, but they also end up in the group's element id.
The id is only used in the configuration UI so it does not create any problems during normal use of the editor.
(Unless maybe the tab makes a difference for screen readers when moving between button groups.)
Proposed resolution
Properly replace all whitespaces using the RegExp /\s/g
instead of the current use of / /g
.
Remaining tasks
None
User interface changes
No visible changes.
API changes
None
Comment | File | Size | Author |
---|---|---|---|
#1 | d8-ckeditor-ui-whitespaces.2350065.1.patch.patch | 817 bytes | TwoD |
Comments
Comment #1
TwoDAnd here's the patch. :)
Comment #3
TwoDThe poor testbot wasn't prepared for DrupalCon. :(
Comment #5
Wim LeersGreat, thanks! :)
Comment #6
alexpottCommitted c752a36 and pushed to 8.0.x. Thanks!
Comment #9
YesCT CreditAttribution: YesCT commentedfixing the tag, so I can delete the least used one.