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

Responsive controls #471

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Responsive controls #471

wants to merge 9 commits into from

Conversation

mman
Copy link
Collaborator

@mman mman commented Feb 6, 2025

This PR closes #470. I chose the following approach:

  • Removed all hardcoded width and height attributes from .svg files to make them resizable as intended.
  • Added --uix CSS variable with default value of 1.0 and going in steps up to 2.0 for 4k displays.
  • Added custom --px-X CSS variables scaling up hard coded pixel values with --uix.
  • Added tailwindcss overrides for spacing and borderWidth using these CSS --px-X variables.
  • Changed the size of Page Selector.
  • Changed the size of Settings, Close, and Back buttons.
  • Changed the size of drill down right arrow button.
  • Changed the size of RadioButton and ToggleSwitch.

Here are the remaining things I would like to address:

  • Verify various other elements for their px hard coded sizes.

@mman mman force-pushed the responsive-controls branch from 0a9161a to 7abd12c Compare February 6, 2025 16:21
@mman mman force-pushed the responsive-controls branch from 7abd12c to de26183 Compare February 17, 2025 17:34
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.

bug: navigation buttons are too small on Garmin 9k
1 participant