Skip to content

UX Refresh - #115

Merged
VicHofs merged 1 commit into
developfrom
feature/ux-refresh
Jan 31, 2026
Merged

UX Refresh#115
VicHofs merged 1 commit into
developfrom
feature/ux-refresh

Conversation

@VicHofs

@VicHofs VicHofs commented Jan 31, 2026

Copy link
Copy Markdown
Contributor

This PR modernizes the app startup and routing UX, adds better offline/error handling, and reorganizes assets for consistency. Changes:

  • Add an animated Lottie splash screen that is controlled from app setup and hides cleanly once data is ready.
  • Introduce a centralized toast host and surface user-facing error messages for setup, routing, search, and location permissions.
  • Add a top-of-screen banner with retry cooldown to handle failed setup and offline states.
  • Fetch buildings + popular destinations with retries and integrate NetInfo for offline detection.
  • Improve marker selection and route lifecycle, including route clearing and camera centering behavior.
  • Rework path drawing to use Reanimated worklet-driven slicing for smoother, more reliable animations.
  • Reorganize assets into assets/images and assets/lottie, updating all references and app config.
  • Harden API query normalization for edge cases around decoding.
  • Minor pointer-events adjustments for the directions legend.
  • Update dependencies (NetInfo, Sonner, Lottie).

Add animated Lottie splash screen with context-controlled hide and splash colors; introduce ToastHost plus toast error handling for setup, routing, search, and location permissions; add offline/setup failure Banner with retry cooldown and timer; fetch buildings and popular destinations with retry + NetInfo offline tracking; rework marker selection, route clearing, and camera centering behavior; replace path draw animation with reanimated worklet-driven slice rendering; reorganize assets into images/lottie directories and update component/app config references; harden API query normalization for unsafe decoding; tweak pointer-events for the directions legend; update deps: add netinfo/sonner, bump lottie
@github-actions
github-actions Bot requested a review from limetheman January 31, 2026 21:38
@VicHofs
VicHofs changed the base branch from main to develop January 31, 2026 21:38
@VicHofs
VicHofs requested a review from BigQise January 31, 2026 21:39

@limetheman limetheman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work! Could possibly abstract some helper functions away from the home page to make it less crowded, and add documentation for helpers/functions (a potential future task). Other than that lgtm

Comment thread app/Home/index.tsx
// toast.error("Couldn't load buildings", {
// id: TOAST_IDS.buildings,
// description: "Check your connection and try again.",
// });

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Could remove this comment

@VicHofs
VicHofs merged commit 724ef2a into develop Jan 31, 2026
1 check passed
VicHofs added a commit that referenced this pull request Feb 9, 2026
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