-
Notifications
You must be signed in to change notification settings - Fork 28
t/1: Introduced Inline Editor #2
Conversation
…efactoring and minor improvements in manual test.
tests/inlineeditorui.js
Outdated
import ClassicTestEditor from '@ckeditor/ckeditor5-core/tests/_utils/classictesteditor'; | ||
|
||
import FocusTracker from '@ckeditor/ckeditor5-utils/src/focustracker'; | ||
import * as enableToolbarKeyboardFocus from '@ckeditor/ckeditor5-ui/src/toolbar/enabletoolbarkeyboardfocus'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't I comment on this in some other PR? I can't find it now ;| tl;dr was that this is unsafe.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, I mistakenly merged the PR in editor-classic which has exactly the same issue: https://github.com/ckeditor/ckeditor5-editor-classic/pull/50/files/7377a8021ddf03236599622e9e768058e21df18f#r104738910. Please, fix both at the same time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed directly in master of editor-classic ckeditor/ckeditor5-editor-classic@8b936cd.
Proposed merge commit message
Feature: Introduced Inline Editor. Closes: ckeditor/ckeditor5#2250.