[Fix] Tibetan renders properly - #647
Conversation
jzohrab
left a comment
There was a problem hiding this comment.
Marking as "requesting changes", due to my lack of understanding. On first read through the idea seems well organized, I just have questions about the backwards compatibility for existing themes people may have.
| --status-4-color: #a5bbe2; | ||
| --status-5-color: #c7b1dd; | ||
|
|
||
| --lute-theme-font-stack: Georgia, "Times New Roman", serif; |
There was a problem hiding this comment.
Hm, is this a potential breakage point? e.g, there's a PR for a new theme, #658 . Also, some users may have their own custom themes (ref https://luteorg.github.io/lute-manual/usage/themes/themes.html#custom-themes).
Is there a way to define this variable at a global level so that new submissions and existing styles will always work?
| @@ -0,0 +1,31 @@ | |||
| /** | |||
There was a problem hiding this comment.
This looks interesting, and I don't know enough about it to comment intelligently. Is this file to be augmented with other scripts for different languages? i.e. different unicode-range entries would get different "handlers".
There was a problem hiding this comment.
Or could/should this be handled by "custom themes", instead of adding new behaviour to the code?
|
|
||
| --border-bottom-color: #ACB2B9; | ||
|
|
||
| --lute-theme-font-stack: Rubik, BlinkMacSystemFont, -apple-system, "Segoe UI", Roboto, Oxygen, Ubuntu, Cantarell, "Fira Sans", "Droid Sans", "Helvetica Neue", Helvetica, Arial, sans-serif; |
There was a problem hiding this comment.
Still learning about this code -- I see in the LingQ theme that there's a body { ... font: .... } block at the top, so this --lute-theme-font-stack is basically duplication of that existing code. Duplication is a sign of a potential code problem, so it would be good to see what options there are for dealing with this.
|
(PRs should go to |
|
Some hacking: adding
.... so probably some kind of custom styling would do in a hack, instead of changing code. But I'm curious as to why I had to do this at all, if @axel584 didn't. |
I also checked again with normal lute and had the same result. Maybe this is needed on mac, but for some reason on @axel584's linux environment it's not needed. @jzohrab Would you suggest a different way to implement the change I did? If so how would you imagine it? |







Summary
Currently Tibetan isn't rendered properly, characters aren't stacking as they are supposed to. This is because the font used doesn't support Tibetan script stacking. This PR addresses that.
Testing
Before (Characters not stacking properly)
After (Correct character stacking)
