Skip to content

Fix mobile nav scrolling and align CI/workflows with WealthTrack - #4

Merged
cruddasj merged 2 commits into
mainfrom
codex/fix-mobile-scrolling-with-nav-menu-open
Mar 18, 2026
Merged

cruddasj merged 2 commits into
mainfrom
codex/fix-mobile-scrolling-with-nav-menu-open

Conversation

@cruddasj

Copy link
Copy Markdown
Owner

Motivation

  • Restore normal vertical scrolling on mobile devices when the navigation menu is open by adjusting the page scroll container behavior.
  • Bring repository automation and contributor guidance in line with the tested workflow set used in WealthTrack to improve linting and PR test coverage reporting.

Description

  • Change main overscroll handling from overscroll-behavior-y: contain to overscroll-behavior-y: auto in src/styles.css so vertical scrolling works while the mobile nav is open.
  • Rebuild the compiled stylesheet with tailwindcss and update the generated assets/styles.css to reflect the fix.
  • Replace the workflows under .github/workflows with the WealthTrack set by adding dependabot-changelog.yml, lint.yml, and pr-test.yml, updating static.yml, and removing pr-version.yml.
  • Update AGENTS.md to match WealthTrack’s Do/Don't guidance while replacing references to "WealthTrack" with "PWA Template".
  • Add changelog placeholders to CHANGELOG.md and assets/changelog.json using [NEXT_VERSION] and [NEXT_DATE] and include brief notes about the scroll fix and workflow updates.

Testing

  • Ran npm run build:css to regenerate assets/styles.css and the command completed successfully.
  • No additional automated unit or E2E tests were executed in this change; workflow files were added/updated so CI lint and coverage jobs will run on future PRs.

Codex Task

@github-actions

Copy link
Copy Markdown
Contributor

Coverage report

St.
Category Percentage Covered / Total
🔴 Statements 21.42% 175/817
🔴 Branches 27.08% 13/48
🔴 Functions 50% 11/22
🔴 Lines 21.42% 175/817

Test suite run success

1 tests passing in 1 suite.

Report generated by 🧪jest coverage report action from 3ce3ed4

@cruddasj
cruddasj merged commit 13b7acb into main Mar 18, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant