Skip to content

UI: bundled Inter font, one colour per meaning, clearer theme button - #56

Merged
joaoguimaraespro merged 1 commit into
mainfrom
feat/ui-font-palette
Oct 9, 2026
Merged

joaoguimaraespro merged 1 commit into
mainfrom
feat/ui-font-palette

Conversation

@joaoguimaraespro

Copy link
Copy Markdown
Owner
  • Inter ships with the app (@fontsource-variable/inter, served from 'self' — CSP unchanged): the same type on Linux, macOS and Windows instead of DejaVu/system fallbacks. Charts use it too.
  • One palette in shared/chart-options: income & net worth green, expenses rose, invested & portfolio value violet, saved teal, references grey; asset classes from the same family. The dashboard's net-balance line is a quiet dashed grey instead of a competing amber.
  • Theme button label/tooltip names the current theme.

Test plan

  • ng build --configuration production, npm test (123 passed with all three)
  • All three merged locally on the production images: E2E journey passes; screenshots reviewed (light, dark, phone)
  • CI

🤖 Generated with Claude Code

… theme button

- Inter (variable) ships with the app, so text looks the same on Linux, macOS and Windows instead of falling back
  to whatever the system has (DejaVu on Linux). Served from 'self': the CSP is unchanged. Charts use it too.
- One palette (shared/chart-options): income and net worth green, expenses rose, invested and portfolio value
  violet, saved teal, references grey. Asset classes come from the same family. The dashboard's net balance line is
  a quiet dashed grey over the bars instead of a competing amber.
- The theme button's label and tooltip name the current theme (System / Light / Dark).
@joaoguimaraespro
joaoguimaraespro merged commit c866946 into main Oct 9, 2026
10 checks passed
@joaoguimaraespro
joaoguimaraespro deleted the feat/ui-font-palette branch October 9, 2026 19:43
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.

1 participant