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

"Customized" Font Size ignored when switching themes #1657

Closed
jerclarke opened this issue Aug 8, 2018 · 5 comments
Closed

"Customized" Font Size ignored when switching themes #1657

jerclarke opened this issue Aug 8, 2018 · 5 comments
Labels

Comments

@jerclarke
Copy link

jerclarke commented Aug 8, 2018

Hi, first thanks for this amazing app! Better than the ones I've paid for!

The main issue I'm having is this very-reproducible (at least for me) bug where switching themes always resets the font size from my custom choice in the preferences.

Typora version: 0.9.9.16.2
macOS: 10.13.6

Reproduction steps

  • Open preferences and set the font size to Customized
  • Set the font size very very small to make this noticeable (9px)
  • Switch to any other theme

Expected

  • New theme loads, customized font applied according to it's relative CSS
  • (I understand that it won't always look the same, but it should be what that theme will render for 9px)

Actual outcome

  • New theme loads with it's default font size
  • Going back to Preferences and editing the Custom font size fixes it
  • Going up+down 1px (back to 9px) makes it show correctly as whatever that theme displays for 9px

I like to switch themes and font sizes a lot, basically whenever I switch from daylight to evening light, and depending on whether I'm using Typora as a writing tool (full screen, large text) or a notes tool (thin column next to another app). This bug really gets in the way of being able to quickly "browse" my installed themes to find a good one for this content. Each time I switch, I need to open preferences and up+down the font size for no reason.

Thanks for trying to find a fix for this. It seems like all you'd need is an extra procedure after theme switching that checks the custom font size and re-asserts the applicable CSS.


Sidenote: Could we get a menu item + keyboard shortcut for Increase text size and Decrease text size?

That's how all my other editors work, usually with CMD++ and CMD+- like the browsers use. It would be so much more comfortable to use than the Preferences window, and in the case of this bug, I could quickly work around the problem.

Let me know if you'd like this request for shortcuts in a separate ticket :)

Thanks!

@jerclarke
Copy link
Author

typora-theme-font-size-demo

@abnerlee abnerlee added the bug label Aug 8, 2018
@abnerlee
Copy link
Contributor

Fixed in new release

@jerclarke
Copy link
Author

Nice, thank you!

I guess I can test it when the new version comes out? Still seeing 0.9.9.17.4 as the version on the site.

@abnerlee
Copy link
Contributor

Does it not working on 0.9.9.17.4 ?

@jerclarke
Copy link
Author

Oh it does work, sorry! I didn't even try that. The changelog made me think this wasn't in the most recent update but I guess you added the fix and it updated itself!

Thanks!

Please still consider adding CMD++ and CMD+- for shrinking/growing the text size without entering the settings :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants