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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: persist lyrics zoom level #1556

Merged
merged 1 commit into from Oct 26, 2022
Merged

feat: persist lyrics zoom level #1556

merged 1 commit into from Oct 26, 2022

Conversation

phanan
Copy link
Member

@phanan phanan commented Oct 26, 2022

No description provided.

@what-the-diff
Copy link

what-the-diff bot commented Oct 26, 2022

  • Added a new preference for lyrics zoom level
  • Set the default value of this preference to 1 (100%)
  • Updated LyricsPane component to use this setting instead of hardcoding it at 100%
  • Snapshot test was updated because we now set font size in rems, not ems anymore and also added style attribute with computed font-size property based on user's preferences

@phanan phanan merged commit 65d5a54 into master Oct 26, 2022
@phanan phanan deleted the feat/save-zoom-lvl branch October 26, 2022 19:24
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.

None yet

1 participant