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

When opening command palette by cmd+shift+p, the cursor is to the left of ">" sign #6011

Closed
gennad opened this issue Aug 21, 2019 · 1 comment · Fixed by #6435
Closed

When opening command palette by cmd+shift+p, the cursor is to the left of ">" sign #6011

gennad opened this issue Aug 21, 2019 · 1 comment · Fixed by #6435
Labels
browser/safari issues related to the safari browser bug bugs found in the application

Comments

@gennad
Copy link
Contributor

gennad commented Aug 21, 2019

Description

When opening command palette by cmd+shift+p, the cursor is to the left of ">" sign.
It should be to the right.

Reproduction Steps

git clone latest Theia.
Follow the instructions to run Theia in a browser.
Open localhost:3000 in Safari.
Press cmd + shift + p.
Observe that cursor is to the left of ">".

OS and Theia version:
Mac OS 10.14.6
Pulled from master, latest commit f46ebf4

@vince-fugnitto
Copy link
Member

That's very odd, I only notice the bug on Safari.

@vince-fugnitto vince-fugnitto added bug bugs found in the application browser/safari issues related to the safari browser labels Aug 21, 2019
vince-fugnitto added a commit that referenced this issue Oct 22, 2019
Fixes #6011

Fixes incorrect command palette cursor position especially on Safari browser.
Added additional handling to verify that if no selection is currently present,
force the cursor at the end of the input. Inspired by a similar implementation
in VS Code.

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
vince-fugnitto added a commit that referenced this issue Nov 1, 2019
Fixes #6011

Fixes incorrect command palette cursor position especially on Safari browser.
Added additional handling to verify that if no selection is currently present,
force the cursor at the end of the input. Inspired by a similar implementation
in VS Code.

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
akosyakov pushed a commit to akosyakov/theia that referenced this issue Feb 24, 2020
Fixes eclipse-theia#6011

Fixes incorrect command palette cursor position especially on Safari browser.
Added additional handling to verify that if no selection is currently present,
force the cursor at the end of the input. Inspired by a similar implementation
in VS Code.

Signed-off-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser/safari issues related to the safari browser bug bugs found in the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants