Skip to content

font list as fallback; click to change font#1

Open
patarapolw wants to merge 6 commits intomarciska:mainfrom
patarapolw:font-list-and-click
Open

font list as fallback; click to change font#1
patarapolw wants to merge 6 commits intomarciska:mainfrom
patarapolw:font-list-and-click

Conversation

@patarapolw
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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