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

LaTeX parser incorrectly recognizes names with diactrics #130

Closed
nightuser opened this issue Dec 7, 2021 · 2 comments
Closed

LaTeX parser incorrectly recognizes names with diactrics #130

nightuser opened this issue Dec 7, 2021 · 2 comments
Assignees
Labels
1-bug 🐛 Issue type: Bug report (something isn't working as expected) 2-confirmed Issue status: Confirmed, reproducible bug in LTeX 3-fixed Issue resolution: Issue has been fixed on the develop branch
Milestone

Comments

@nightuser
Copy link

Describe the bug
If a name have several diactrics, only the part of it is recognized as a word.

Steps to reproduce
Steps to reproduce the behavior:

  1. Have a name Žák in the text, written as {\v Z}{\'a}k.
  2. ltex parses it as “ák” and produces the error “'ák': Possible spelling mistake found.”

Expected behavior
It should be recognized correctly.

LTeX configuration
Default.

Version information
List here the version information of the relevant software.

  • Operating system: ArchLinux
  • ltex-ls: 15.2
  • Java: openjdk 17.0.1
@nightuser nightuser added 1-bug 🐛 Issue type: Bug report (something isn't working as expected) 2-unconfirmed Issue status: Bug that needs to be reproduced (all new bugs have this label) labels Dec 7, 2021
@valentjn
Copy link
Owner

valentjn commented Dec 8, 2021

It works when using \v{Z}\'{a}k, or when using Unicode (LuaLATEX).

I'll have to see whether this will be fixed. Formally, { ... } is opening a new group, which could contain anything, really (not just a single letter), and it's pretty plain-TEX-ish.

@valentjn valentjn added 2-confirmed Issue status: Confirmed, reproducible bug in LTeX 3-fixed Issue resolution: Issue has been fixed on the develop branch and removed 2-unconfirmed Issue status: Bug that needs to be reproduced (all new bugs have this label) labels Dec 12, 2021
@valentjn valentjn added this to the 15.3.0 milestone Dec 12, 2021
@valentjn valentjn self-assigned this Dec 12, 2021
@github-actions
Copy link

This issue is now fixed on develop. The fix will be included in the next release of LTEX.

If you don't want to wait, you can try out the nightly pre-release tomorrow. Nightly pre-releases are published every morning at around 4am UTC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1-bug 🐛 Issue type: Bug report (something isn't working as expected) 2-confirmed Issue status: Confirmed, reproducible bug in LTeX 3-fixed Issue resolution: Issue has been fixed on the develop branch
Projects
Status: 🎉 Done
Development

No branches or pull requests

2 participants