Skip to content

[NAV-002] Add route transition progress bar - #66

Merged
chev0004 merged 2 commits into
developfrom
feat/nav-002
Jun 20, 2026
Merged

[NAV-002] Add route transition progress bar#66
chev0004 merged 2 commits into
developfrom
feat/nav-002

Conversation

@chev0004

@chev0004 chev0004 commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Outline

Summary

Adds a reusable top-of-screen route transition progress bar for localized app navigation. The PR also replaces the toast viewport wrapper to avoid the Radix ref update loop that caused the Discovery page runtime error.


Changes & Enhancements

  • Type of Change:
    • Feature
    • Bugfix
    • Refactor
    • Chore
  • Changes:
    • Navigation:
      • Added a route progress provider with pathname completion, guarded fallback cleanup, and primary-accent styling.
    • Routes:
      • Routed navbar, menu, language switcher, onboarding, profile, saved, settings, and discovery client navigation through the progress-aware router.
    • Toast:
      • Replaced the Radix toast viewport dependency with the app's native toast stack markup to prevent the maximum update depth crash.

Page / App Changes

Affected Next.js Routes (Local):
/en
/en/profile
/en/saved
/en/settings
/en/onboarding

Before (Screenshot)
After (Screenshot)

Component / Storybook Changes

Affected Storybook Components (Local):

  • Components/Toast > Default
  • Components/Toast > Auto Dismiss
  • Features/Inbox > Live Update
Before (Screenshot)
After (Screenshot)

Not Doing


Other (Remarks)


Checklists

  • Code is well-documented (comments, JSDoc, etc.).
  • Existing comments were updated as needed.
  • No out-of-scope changes are included.
  • Write TODO comments where future work is required.
  • Removed unnecessary debug code (e.g., console.log, debugger).
  • Self-reviewed and tested locally.

@chev0004 chev0004 self-assigned this Jun 20, 2026
@chev0004
chev0004 merged commit 1a74171 into develop Jun 20, 2026
1 check passed
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