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

Emoji keyboard opens in the wrong place #13093

Closed
roblourens opened this issue Sep 30, 2016 · 9 comments
Closed

Emoji keyboard opens in the wrong place #13093

roblourens opened this issue Sep 30, 2016 · 9 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug editor-core Editor basic functionality editor-input Editor text input verified Verification succeeded
Milestone

Comments

@roblourens
Copy link
Member

Ctrl+cmd+space to open this on OSX. Sometimes it's on the wrong monitor, sometimes it's fairly close like in the gif. Everywhere else in the system, it opens pointing at the cursor or focused input box, so I'm guessing it's something we actually have control over somehow. If the cursor is in a basic input box like the find widget, marketplace search box, etc, then it shows up in the right place.

sep-30-2016 11-10-27

@alexdima
Copy link
Member

alexdima commented Oct 6, 2016

It shows at the top of the editor, where we keep the textarea

@alexdima alexdima added bug Issue identified by VS Code Team member as probable bug editor-core Editor basic functionality labels Oct 6, 2016
@Tyriar
Copy link
Member

Tyriar commented Oct 6, 2016

@alexandrudima do you move the text area under the cursor when doing an IME composition? I was expecting an emoji IME to act the same as a language IME.

@roblourens
Copy link
Member Author

In the gif that appears to be the case but I see it in different places depending on where the cursor is.

@alexdima
Copy link
Member

alexdima commented Oct 6, 2016

I move the textarea to the cursor location when I get a compositionstart event. Chromium/Electron doesn't give a compositionstart event when popping up the emoji picker

@mushanshitiancai
Copy link

I meet similar problem with Chinese IME.

image

@cesarvarela
Copy link

image

Not just chinese

@alexdima alexdima added the editor-input Editor text input label Mar 2, 2017
@alexdima alexdima added this to the Backlog milestone Mar 2, 2017
@rebornix rebornix self-assigned this Apr 27, 2017
@alexdima
Copy link
Member

alexdima commented May 5, 2017

After ad023e4 and 689a607 (done for #356) here is the status:

@mushanshitiancai Chinese IME is tracked in #356 . The first time, the suggestions pop up at the correct location. On a second usage, the suggestions pop up at the top left window position. This appears now to be an upstream bug which I've filed against Electron.

@cesarvarela This now works correctly:
accents-good-popup

@alexdima
Copy link
Member

alexdima commented May 5, 2017

After my latest change, the emoji picker shows "really close" to the primary cursor. I can only guess that the emoji picker is positioned in a way that does not account for the overflow: hidden of the textarea and tries really hard (but fails) to position itself next to the native cursor inside the textarea:

emoji-picker

@alexdima alexdima modified the milestones: May 2017, Backlog May 5, 2017
@roblourens
Copy link
Member Author

What I see now is that it's always right on the cursor but maybe a little too close for comfort. Huge improvement though! Opening a new issue.

@roblourens roblourens added the verified Verification succeeded label Jun 1, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug editor-core Editor basic functionality editor-input Editor text input verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

6 participants