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 commentedfixing the tag, so I can delete the least used one.