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

[Bugs - Geman extension] Cut works when use the dieresis #1321

Closed
rbalet opened this issue Feb 4, 2019 · 4 comments · Fixed by #1322
Closed

[Bugs - Geman extension] Cut works when use the dieresis #1321

rbalet opened this issue Feb 4, 2019 · 4 comments · Fixed by #1322

Comments

@rbalet
Copy link

rbalet commented Feb 4, 2019

Seems to cut the words in 2 when use the dieresis
image

@adufr
Copy link

adufr commented Feb 12, 2020

I'm still experiencing this issue as of today

@rbalet rbalet changed the title [Bugs - Geman extension] Cut works when us the dieresis [Bugs - Geman extension] Cut works when use the dieresis Feb 12, 2020
@Jason3S
Copy link
Collaborator

Jason3S commented Nov 9, 2020

@woosy would you please give some sample text that I can copy / paste? It might be a UTF-8 issue.

@Jason3S Jason3S transferred this issue from streetsidesoftware/vscode-cspell-dict-extensions Jun 8, 2021
@Jason3S
Copy link
Collaborator

Jason3S commented Jun 8, 2021

It looks like cspell is NOT correctly handling text like geschäft when the a and ̈ dieresis are not a single letter.
It does work when it is a single letter: ä.

Jason3S added a commit that referenced this issue Jun 8, 2021
@Jason3S Jason3S linked a pull request Jun 8, 2021 that will close this issue
Jason3S added a commit that referenced this issue Jun 9, 2021
* Fix: #1321
* refactor: minor refactor of Reader.ts
@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 14, 2021
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 a pull request may close this issue.

3 participants