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

Removed the confirmation of mention selection by using space #92

Closed
wants to merge 2 commits into from

Conversation

fredck
Copy link
Contributor

@fredck fredck commented Mar 9, 2020

Suggested merge commit message (convention)

Other: The space and tab keys are not anymore used to confirm a mention selection from the list. Closes ckeditor/ckeditor5#6394.

@fredck
Copy link
Contributor Author

fredck commented Mar 9, 2020

The Travis checks seem to have failed because of issues not related to this PR. Got all tests green on my side.

@fredck
Copy link
Contributor Author

fredck commented Mar 21, 2020

Force-pushed a rebase over v18.0.0.

@fredck fredck changed the title Removed the confirmation of mention selection by using the space and tab Removed the confirmation of mention selection by using space Mar 30, 2020
@fredck
Copy link
Contributor Author

fredck commented Mar 30, 2020

The tab key is indeed a valid key for mention confirmation as it is commonly used in programming IDEs for that purpose. Additionally, unlike space, tab is mostly never used as a common typing key after mention text. 

Because of the above I've reverted the changes related to the tab key, reducing the scope of the changes to the space key only.

@fredck fredck force-pushed the t/ckeditor5/6394 branch from ef62d5a to 3871e1f Compare May 2, 2020 12:51
@fredck
Copy link
Contributor Author

fredck commented May 2, 2020

Force-pushed a rebase over v19.0.0.

@zadam
Copy link

zadam commented May 9, 2020

It would be great to see this fix in the next release!

@fredck
Copy link
Contributor Author

fredck commented May 27, 2020

The PR has been moved to ckeditor/ckeditor5#7306

For those interested, be sure to add 👍 and further feedback there.

@fredck fredck closed this May 27, 2020
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.

Mention: do not confirm selection on "space"
2 participants