Marek Lewandowski
Marek Lewandowski
The [https://github.com/ckeditor/ckeditor5/issues/12178#issuecomment-1205363543](https://github.com/ckeditor/ckeditor5/issues/12178#issuecomment-1205363543) should be fixed by #12229 (on the latest master).
While working on this issue we can look how we solve this problem in CKE4. We could add a config property that would allow for providing a human readable title...
The difference is that during the (this side) button's `execute` event the toolbar is already hidden and focus is brought back to the button itself. So `this.panelView.isVisible` is evaluating to...
Indeed the issue came from #12003 In a52516a we removed the prevent default method call for click event. This causes the button to gain focus and panel to close prematurely.
A patch that fixes (this particular issue) is in [https://github.com/ckeditor/ckeditor5/tree/ck/12184-closing-balloon-block-toolbar-focus](https://github.com/ckeditor/ckeditor5/tree/ck/12184-closing-balloon-block-toolbar-focus)
Mentioned in a "column width" row of [https://github.com/ckeditor/ckeditor5/issues/5652#issuecomment-545472596](https://github.com/ckeditor/ckeditor5/issues/5652#issuecomment-545472596) so let's keep it in sync in case we fix it. I checked quickly what markup is placed on Chrome (Win10) to...
@ckeditor/qa-team when tested please give R+ using GH's review system so that it's clearly visible for us 🙂 I believe that R+ from one of you is enough in this...
Couple of questions: * What's exactly wrong with the current focus indication? Is it too thin, is it a matter of color, is it a matter of conveying information by...
@oleq can you have a look at it?
A way that CKE4 mitigates this issue is by adding an extra outline to the button. Let's see what are the options to satisfy this requirement. See: * [https://www.w3.org/WAI/WCAG21/Understanding/non-text-contrast.html](https://www.w3.org/WAI/WCAG21/Understanding/non-text-contrast.html) *...