Skip to content

Task 6: Map Loading States & Shared Error Toast Utility#886

Merged
mraula merged 4 commits into
mainfrom
EHMI/VG/toast-utility
May 8, 2026
Merged

Task 6: Map Loading States & Shared Error Toast Utility#886
mraula merged 4 commits into
mainfrom
EHMI/VG/toast-utility

Conversation

@Vamshi-Gollapelly
Copy link
Copy Markdown
Collaborator

This PR implements Task 6 for the HMI map page.

Changes made:

  • Added a reusable shared toast notification utility in HMI-utils.js
  • Added a loading spinner overlay for the map while data is loading
  • Added an error banner with a retry button when map data fails to load
  • Added API timeout handling in routes.js so requests fail clearly instead of hanging
  • Improved map page behaviour so it no longer goes blank on load failure

Files updated:

  • src/Components/HMI/ui/public/index.html
  • src/Components/HMI/ui/public/js/HMI-utils.js
  • src/Components/HMI/ui/public/js/HMI.js
  • src/Components/HMI/ui/public/js/routes.js

How to test:

  1. Open the HMI page
  2. Confirm the spinner appears while the map is loading
  3. Confirm the toast utility shows notifications correctly
  4. Simulate API/backend failure and confirm the error banner appears
  5. Click Retry and confirm the app attempts to load again
  6. Confirm requests timeout instead of hanging indefinitely

Outcome:
This update adds consistent user feedback for loading and failure states and provides a reusable toast utility for other team members to use.

@Vamshi-Gollapelly
Copy link
Copy Markdown
Collaborator Author

Hi, I’ve completed Task 6 (Map Loading States & Toast Utility).
Could you please review and let me know if any changes are needed?

Copy link
Copy Markdown
Collaborator

@mraula mraula left a comment

Choose a reason for hiding this comment

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

everything looks good

@mraula mraula merged commit fbdd3e8 into main May 8, 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.

2 participants