Dariusz Jarzębski
Dariusz Jarzębski
~Sorry, one more thing - one unit test fails for me locally: `updates view#height & view#width on DOM "mouseover" event`, but I see that CI is green and it works...
2 notes after debugging this issue: ### Reproduction steps clarification It won't occur unless the image toolbar _replaces_ the link actions view on the inserted image element. Issue will occur...
A cheap and dirty idea could be checking the `document.activeElement` when dropdown is being closed and don’t focus the button if editable is focused, but I’m not sure if it...
PoC: `ck/12178-grouped-toolbar-focus`. cc @Reinmar @mlewand
Most of the issues above seem to be the same as #12207, only the reproduction steps are a bit different (using toolbar buttons) - @FilipTokarski @Acrophost please verify it and...
Yes, the image button doesn't focus the editable - https://github.com/ckeditor/ckeditor5/blob/master/packages/ckeditor5-image/src/imageupload/imageuploadui.js#L57-L63. I think it will be worth an actual issue.
Thanks QA, I'll create a PR with the fix.
> This sounds to me like a result of focus management brought in [#12162](https://github.com/ckeditor/ckeditor5/pull/12162) I'd be surprised if that's true as we only touched the dropdown behaviour. I'll check if...
I checked the manual test http://localhost:8125/ckeditor5-build-balloon-block/tests/manual/ckeditor.html at the a0244b0 commit (so before all commits from [#12162](https://github.com/ckeditor/ckeditor5/pull/12162)) and this problem is already present there.