Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed the confirmation of mention selection by using the space key #7306

Merged
merged 3 commits into from
Aug 18, 2020

Conversation

fredck
Copy link
Contributor

@fredck fredck commented May 27, 2020

Suggested merge commit message (convention)

Other (mention): The space keys is not anymore used to confirm a mention selection from the list. Closes #6394.


Additional information

Original PR: ckeditor/ckeditor5-mention#92

@ghost
Copy link

ghost commented Jul 22, 2020

@fredck
Why not doing the same with keyCodes.space as with keyCodes.esc, calling this._hideUIAndRemoveMarker();? e.g. in github when using the @, if you enter space after the @, the menu goes away....

@fredck
Copy link
Contributor Author

fredck commented Aug 11, 2020

Why not doing the same with keyCodes.space as with keyCodes.esc, calling this._hideUIAndRemoveMarker();? e.g. in github when using the @, if you enter space after the @, the menu goes away....

@pammann-work, I'm not sure I've understood this. With the provided patch, if you enter space after @ the menu goes away, exactly as you described. In fact, the final behavior will match exactly what you see in GitHub. Am I missing something?

@ghost
Copy link

ghost commented Aug 11, 2020

@fredck, good to hear to it will work like in GitHub. I was just looking at the code changes and did not try it out. Thanks for your confirmation.

@jodator jodator self-assigned this Aug 18, 2020
@jodator jodator merged commit a8d41ec into master Aug 18, 2020
@jodator jodator deleted the i/6394 branch August 18, 2020 07:28
@robjbrain
Copy link

I came to this issue by following a path of closed issues
#4665 (comment)
ckeditor/ckeditor5-mention#92 (comment)

The original request was that we be able to mention names that have a space in them e.g. "John Doe" rather than closing the mention dialog after typing the space in "John ".

I can see that this change was merged and I can see the updated code in my node_modules directory so i'm definitely up to date but the functionality is still the same, the menu is closed when a space is typed.

Is there a solution that prevents the menu from closing on a space?

@EricPaulson
Copy link

This PR does not solve the problem. As soon as I press the spacebar, the mention dropdown disappears. This is not how any user would expect this feature to work.

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

Successfully merging this pull request may close these issues.

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