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

CJK charaters are being rendered as squares #128

Closed
Guanran928 opened this issue Apr 6, 2024 · 7 comments · Fixed by #131
Closed

CJK charaters are being rendered as squares #128

Guanran928 opened this issue Apr 6, 2024 · 7 comments · Fixed by #131
Assignees
Labels
bug Something isn't working rust Pull requests that update Rust code

Comments

@Guanran928
Copy link

I'm using Source Han Sans VF as CJK font, does this mean centerpiece don't support font fallback?
image

@Guanran928
Copy link
Author

#103 (comment), oops, didn't search with the right keyword

Will close when #81 is merged

@friedow friedow added bug Something isn't working rust Pull requests that update Rust code labels Apr 13, 2024
@friedow
Copy link
Owner

friedow commented Apr 15, 2024

I've enabled font fallback in #131. Feel free to test that branch and leave a review whether this works for you :).

@friedow friedow self-assigned this Apr 15, 2024
@Guanran928
Copy link
Author

Hmm, I still cant see Chinese in window titles

❯ nix run github:friedow/centerpiece/pull/131/head -- --version
centerpiece 1.0.0 - 2024-04-15 - 7318ea3

@Guanran928
Copy link
Author

Guanran928 commented Apr 15, 2024

Oh, you can see CJK & emojis in the search box (weirdly, also in 0.8.0 from nixpkgs), but not in the selector.
image

@friedow
Copy link
Owner

friedow commented Apr 19, 2024

That's weird. I've ran the same command:

> nix run github:friedow/centerpiece/pull/131/head -- --version
centerpiece 1.0.0 - 2024-04-15 - 7318ea3

> nix run github:friedow/centerpiece/pull/131/head

The title of this website: https://www.archchinese.com/ is displayed correctly in the brave history plugin:
image

Can you check whether the characters work in the brave history plugin for you?

(FYI: The version in nixpkgs had advanced text shaping in the input box but not in the entry list. This is why it looks that way ;))

@Guanran928
Copy link
Author

Guanran928 commented Apr 21, 2024

Yeah, it's working fine now. I guess maybe I ran the wrong version last time?

@friedow
Copy link
Owner

friedow commented Apr 21, 2024

Oh perfect, I'm really happy to hear that :). Will merge the Bugfix branch then!

friedow added a commit that referenced this issue Apr 21, 2024
This fixes #103 and it should also fix #128.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working rust Pull requests that update Rust code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants