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

font list as fallback; click to change font #1

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

patarapolw
Copy link

@patarapolw patarapolw commented Jun 9, 2023

Canvas support isn't yet guaranteed. Checking for browser is an option, but there are probably better choices.

At least the fallback isn't always the default font. Before fix, sometimes it is a mixed between default font and random font.

Currently, font is sometimes mixed.

image

Click to change font is an additional feature, rather than a fix.

Force a new random font with shortcut CTRL+J during review.

CTRL+J opens Download in Brave Browser, making it bad for changing fonts.

@marciska
Copy link
Owner

marciska commented Oct 5, 2024

Do you want to retarget your PR to main branch together with the new features? :)

@patarapolw patarapolw changed the base branch from dev to main October 9, 2024 09:17
@patarapolw
Copy link
Author

I am not using WaniKani currently anyway, so not sure what would be broken.

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.

2 participants