Skip to content

add font size modifier to top bar - #121

Open
cssma wants to merge 1 commit into
mainfrom
feature-font-size
Open

add font size modifier to top bar#121
cssma wants to merge 1 commit into
mainfrom
feature-font-size

Conversation

@cssma

@cssma cssma commented May 17, 2026

Copy link
Copy Markdown
Collaborator

📄 Pull Request Description

Please fill out each section below.
The goal is that reviewers can understand everything without reading the code.


🧩 What was changed?

Added a font size modifier slider to the top bar, next to the existing contrast slider.


💡 Why was it changed?

[Frontend] Add font size modifier
JUCanEat/documentation#72


⚙️ How was it implemented?

A font size slider (100% - 130%) was added to the top bar. The selected font size is applied to the html element and persisted in localStorage. The top bar title hides automatically when there is not enough space.

⚠️ Side Effects or Risks

Increasing font size scales the entire app including the top bar. The title collapses when it would overflow.


✅ Checklist

  • All 4 sections above are clearly filled out
  • Tests and documentation updated

@cssma
cssma requested a review from kon-mtal May 17, 2026 14:25

@kon-mtal kon-mtal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, haven't tested with backend up because I couldn't get it to run.

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