Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

t/181: Feature: The Ctrl+K keystroke should open link URL editing dialog #190

Merged
merged 1 commit into from
Apr 5, 2018

Conversation

oleq
Copy link
Member

@oleq oleq commented Apr 4, 2018

Suggested merge commit message (convention)

Feature: The Ctrl+K keystroke should open link URL editing dialog. Closes ckeditor/ckeditor5#4825.

@oleq oleq requested a review from dkonopka April 4, 2018 15:00
@coveralls
Copy link

coveralls commented Apr 4, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 3ffef6f on t/181 into 711e591 on master.

Copy link
Contributor

@dkonopka dkonopka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ctrl+K is not opening editing dialog if one of action button (url, edit, unlink) is focused. In case without focus it works properly.

Maybe we need an additional test here to check keystroke with focused button.

@oleq
Copy link
Member Author

oleq commented Apr 5, 2018

kapture 2018-04-05 at 10 31 21

WFM. But I noticed that it does not work with Cmd+K but with Ctrl+K it is OK. Is that what concerns you?

@dkonopka
Copy link
Contributor

dkonopka commented Apr 5, 2018

@oleq Right, exactly what I've meant. With Ctrl+K it's ok. If it shouldn't work with Cmd then everything is ok.

@oleq
Copy link
Member Author

oleq commented Apr 5, 2018

It's a different issue. See https://github.com/ckeditor/ckeditor5-utils/issues/234.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ctrl+K should open link URL editing dialog
3 participants