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

fix(pdf-service): fix bug with German umlauts not rendered correctly in pdf #1535

Merged
merged 4 commits into from
Dec 9, 2024

Conversation

rbrtrfl
Copy link
Contributor

@rbrtrfl rbrtrfl commented Dec 9, 2024

I could only reproduce this bug with the arc browser. The root cause were two titles in bargeld and wertsachen where the 'ü' of Eigentümer:in was written as a NFD (normalized form canonical decomposition) which means the 'u' was typed first, followed by the combining diacritical marks. pdfkit does not seem to properly support encoding these characters in few cases (arc browser).
Proposal: Added a linter rule that detects NFDs.

https://app.asana.com/0/1203259475859667/1208834671333919

Copy link

sonarqubecloud bot commented Dec 9, 2024

Copy link
Contributor

@aaschlote aaschlote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Out of curiosity, what are arc browsers? Any example?

@rbrtrfl
Copy link
Contributor Author

rbrtrfl commented Dec 9, 2024

LGTM!

Out of curiosity, what are arc browsers? Any example?

There is the one arc browser. Never heard of it before this bug: https://arc.net/

Copy link

sonarqubecloud bot commented Dec 9, 2024

@rbrtrfl rbrtrfl merged commit a7b885f into main Dec 9, 2024
21 checks passed
@rbrtrfl rbrtrfl deleted the pdf-umlaut-bug branch December 9, 2024 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants