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

[OSX][QWERTY] Emphasized character preceded by the unaccented character #7419

Closed
Arubinu opened this issue Sep 29, 2018 · 12 comments · Fixed by matrix-org/matrix-react-sdk#2202
Labels
P1 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Platform-Specific Z-Upstream

Comments

@Arubinu
Copy link

Arubinu commented Sep 29, 2018

Description

Hello, I use a Macbook Pro with Qwerty keyboard. When I want to write an accented character, a box offers me to choose after a long press. The trouble is that the accented character added later is preceded by the character without accent.

Steps to reproduce

  • Long press on the E key: displays the selection box of the accent (possibility to use the numbers of the keyboard to choose).
  • Before the choice, the letter is already displayed.
  • Once the letter É chosen, it is added to the continuation, example in French: Ce texte comporte des caractres accentus et chacun est doubl chaque fois !

In general, the unaccented character is not displayed at all (so it is not deleted later), which makes no duplication.

accents

Version information

  • Platform: web (in-browser) and desktop

For the web app:

  • Browser: Vivaldi 2.0.1309.29 (Stable channel) (64 bits)
  • OS: macOS 10.13 & macOS 10.14
  • URL: riot.im/app 0.16.4 & 0.16.5

For the desktop app:

  • OS: macOS 10.13 & macOS 10.14
  • Version: 0.16.4 & 0.16.5
@lampholder
Copy link
Member

Even worse, the accented character seems to be appended to the message text, after the caret (so you'd have to navigate to the end of the text manually to continue typing).

@aaronraimist
Copy link
Collaborator

Working on the fix for this

@lampholder
Copy link
Member

Excellent news :) In case it's useful (though at this stage I imagine it is not), this problem is not inherent to draft.js (the demo draft.js at https://draftjs.org/ works fine).

@lampholder
Copy link
Member

Oh, and just to save anyone else who's looking into this a Google - if like me you'd turned this feature off so long ago you'd forgotten it existed, you can toggle it with

defaults write -g ApplePressAndHoldEnabled -bool false

(or true, obvs)

@Arubinu
Copy link
Author

Arubinu commented Oct 1, 2018

It still does not work for me with the latest version (riot-web: 0.16.5, olm: 2.2.1).

@aaronraimist
Copy link
Collaborator

aaronraimist commented Oct 21, 2018

This issue can be closed, fixed now in v0.17.0 and later.

@turt2live
Copy link
Member

@Arubinu @lampholder are you able to confirm if this is fixed? I don't have a mac to test with unfortunately.

@turt2live turt2live added the X-Needs-Info This issue is blocked awaiting information from the reporter label Oct 31, 2018
@turt2live turt2live self-assigned this Oct 31, 2018
@lampholder
Copy link
Member

I'm afraid this still seems to be broken on latest /develop (ca30539-react-cf31f01c8584-js-62b2c07be2fa):

ee

@aaronraimist
Copy link
Collaborator

Ohh 😟
It got fixed in Firefox and Safari but is still broken in Chrome

@aaronraimist
Copy link
Collaborator

?? I could have sworn it wasn't an upstream bug before but today I can reproduce it on slatejs.org in Firefox and Chrome (but not Safari)

@turt2live
Copy link
Member

I'm going to trust that this is an upstream issue - if it's not, well, bleh :(

@turt2live turt2live removed the X-Needs-Info This issue is blocked awaiting information from the reporter label Nov 5, 2018
@turt2live turt2live removed their assignment Nov 5, 2018
@turt2live
Copy link
Member

Closing in favour of #7665 to track the upstream issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Platform-Specific Z-Upstream
Projects
None yet
4 participants