Font missing diacritics characters #135

Closed
opened 2025-11-01 20:48:18 -05:00 by GiteaMirror · 7 comments
Owner

Originally created by @sq3tle on GitHub (Apr 24, 2023).

Originally assigned to: @dpschen on GitHub.

Description

Font used in sidebar and task editing panel is missing diacritics characters used in Polish language. In case of some languages, it may be worth to fallback to some regular font.

Vikunja Frontend Version

0.20.5

Vikunja API Version

0.20.4

Browser and version

chrome 112.0.5615.138

Can you reproduce the bug on the Vikunja demo site?

Yes

Screenshots

image

Originally created by @sq3tle on GitHub (Apr 24, 2023). Originally assigned to: @dpschen on GitHub. ### Description Font used in sidebar and task editing panel is missing diacritics characters used in Polish language. In case of some languages, it may be worth to fallback to some regular font. ### Vikunja Frontend Version 0.20.5 ### Vikunja API Version 0.20.4 ### Browser and version chrome 112.0.5615.138 ### Can you reproduce the bug on the Vikunja demo site? Yes ### Screenshots ![image](https://user-images.githubusercontent.com/58919648/233987231-9df85e36-e5d7-473f-9b55-e0bfa3b73138.png)
Author
Owner

@dpschen commented on GitHub (Apr 24, 2023):

Seems like our font subsetting script needs to be extended to support latin-ext. One way could be to extend the SUBSETTER_ARGS argument of the instance_and_subset function to support an array of subsetting arguments. For each element of the array we would create a new output file.

Looking at Googles subsets of OpenSans I'm pretty sure, that there are currently similar problems with our Vietnamese translations.

We need to adjust the preloading of the font as well based on the selected language

@dpschen commented on GitHub (Apr 24, 2023): Seems like our [font subsetting script](https://kolaente.dev/vikunja/frontend/src/branch/main/scripts/fonts-subset.sh#L41) needs to be extended to support `latin-ext`. One way could be to extend the `SUBSETTER_ARGS` argument of the `instance_and_subset` function to support an array of subsetting arguments. For each element of the array we would create a new output file. Looking at [Googles subsets of OpenSans](https://fonts.googleapis.com/css2?family=Open+Sans) I'm pretty sure, that there are currently similar problems with our Vietnamese translations. We need to adjust the preloading of the font as well based on the selected language
Author
Owner

@orhid commented on GitHub (Apr 11, 2025):

has there been any movement on this bug?

@orhid commented on GitHub (Apr 11, 2025): has there been any movement on this bug?
Author
Owner

@dpschen commented on GitHub (Apr 15, 2025):

has there been any movement on this bug?

No, but I try to fix this soon (will try this week). Since our internationalisation has progressed a lot this became more urgent.

@dpschen commented on GitHub (Apr 15, 2025): > has there been any movement on this bug? No, but I try to fix this soon (will try this week). Since our internationalisation has progressed a lot this became more urgent.
Author
Owner

@sceptri commented on GitHub (Jul 16, 2025):

Hey, just wanted to ask... any update on this bug?

@sceptri commented on GitHub (Jul 16, 2025): Hey, just wanted to ask... any update on this bug?
Author
Owner

@kolaente commented on GitHub (Jul 17, 2025):

I've adjusted this in aea2f708d3 - please check with the next unstable build (should be ready for deployment in ~30min, also on try).

@dpschen This was the fastest way to fix the problem here, feel free to optimize this further.

@kolaente commented on GitHub (Jul 17, 2025): I've adjusted this in https://github.com/go-vikunja/vikunja/commit/aea2f708d34ef0b0a87caefaeb842a1c0bbf1b83 - please check with the next unstable build (should be ready for deployment in ~30min, also on [try](https://try.vikunja.io)). @dpschen This was the fastest way to fix the problem here, feel free to optimize this further.
Author
Owner

@dpschen commented on GitHub (Jul 17, 2025):

All good. Sry that i didn’t get to this yet.

@kolaente I‘m a bit confused that you didn’t need to change the unicode range.

@dpschen commented on GitHub (Jul 17, 2025): All good. Sry that i didn’t get to this yet. @kolaente I‘m a bit confused that you didn’t need to change the unicode range.
Author
Owner

@kolaente commented on GitHub (Jul 17, 2025):

@dpschen I did: 3b8258d57e

@kolaente commented on GitHub (Jul 17, 2025): @dpschen I did: https://github.com/go-vikunja/vikunja/commit/3b8258d57ed011a00bb057ed9394f13418246da6
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#135