This is in somewhat gray area. At the time I type this, the only official WCAG target size requirement is 2.5.5 Target Size, and this is an AAA requirement, higher than the AA standard Drupal is built to meet. This requires the clickable element to be at least 44x44px.
However, there is a slightly less strict AA version of this proposed for the next set of WCAG standards, 2.2: 2.5.8 Pointer Target Spacing. The element can be smaller than 44x44 as long as the surrounding whitespace (or anything non-clickable) provides a minimum 44x44 "box" around it
Since some version of 2.5.8 Pointer Target Spacing is likely to be part of the next version of WCAG, and because it benefits tablet/touchscreen users I'd like to add this. The buttons + spacing are currently 36x36px, so an extra 8px of spacing and/or button size would take care of this.
Likely add Drupal specific styles to enlarge the button and/or its spacing, but it's possible this is something the CKEditor devs would want to address and make part of their default theme.
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
bnjmnmOpened a PR for this https://github.com/zrpnr/ckeditor5-drupal-admin/pull/3
Comment #3
bnjmnmSetting to fixed - the PR on the Github repo was merged.
Comment #4
zrpnrThanks for doing this research @bnjmnm, I more or less copied the styles for the admin UI from the CK5 online builder.
That's out of scope here I think, the toolbar UI is totally under our control and sizing the buttons is in css/toolbar.admin.css
We need to make your changes there as well.
Comment #6
zrpnrUpdated toolbar.admin.css with @bnjmnm changes in https://github.com/zrpnr/ckeditor5-drupal-admin/pull/3/files
Comment #8
zrpnrThis seems pretty safe to merge :)