Skip to content

feat: Fitness and Health fit the window with one view at a time - #63

Open
atharva-acharya wants to merge 1 commit into
mainfrom
feat/fitness-health-fit
Open

atharva-acharya wants to merge 1 commit into
mainfrom
feat/fitness-health-fit

Conversation

@atharva-acharya

Copy link
Copy Markdown
Owner

Fitness and Health no longer scroll as a page. Each shows one view at a time, chosen with a segmented control under the title:

  • Fitness: This week, Steps and protein, Weight, Workouts, Lifts.
  • Health: Today, Trends, Workouts.

Every view stays mounted, so switching is instant and charts keep their state; a view taller than the window scrolls inside itself. The page titles lose their eyebrow labels, as on the other redesigned pages. With the ring fix from #59, Health's rings are no longer clipped.

Checked in the real app at 1440 x 900 (windowed) and 1655 x 900 (maximised): no page scroll on either page. Unit tests pass (the Health tab-order test now checks the page tabs and the range tabs separately); e2e opens each view before checking it.

This branch has not been deployed

No deployments
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