Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI: Enhance service alerts display and journey card layout #318

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

ksharma-xyz
Copy link
Owner

@ksharma-xyz ksharma-xyz commented Nov 7, 2024

TL;DR

Enhanced service alert visibility in journey cards and improved UI interactions
Only display journey leg duration when there are multiple transport legs

What changed?

  • Added a counter for unique service alerts in the journey card
  • Redesigned alert button to show count and use a pill-shaped background
  • Adjusted padding and click areas for better touch targets
  • Only display journey leg duration when there are multiple transport legs
  • Added pluralization support for alert count messaging
  • Moved API call to IO dispatcher for better performance
  • Fixed padding in service alert screen

Why make this change?

To improve user awareness of service disruptions and make the interface more intuitive. The changes make it easier to identify journeys with alerts and provide better visual feedback for interactive elements.

Screenshots

Screen.Recording.2024-11-08.at.2.20.55.am.mov

Copy link
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on Graphite Graphite

@ksharma-xyz ksharma-xyz changed the title Update UI: Enhance service alerts display and journey card layout Nov 7, 2024
@ksharma-xyz ksharma-xyz marked this pull request as ready for review November 7, 2024 15:20
@ksharma-xyz ksharma-xyz added the UI Design System Related changes label Nov 7, 2024
@ksharma-xyz ksharma-xyz merged commit 0f7e8ec into main Nov 7, 2024
3 checks passed
Copy link
Owner Author

Merge activity

  • Nov 7, 10:28 AM EST: A user merged this pull request with Graphite.

@ksharma-xyz ksharma-xyz deleted the 11-08-update branch November 7, 2024 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI Design System Related changes
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant