-
Notifications
You must be signed in to change notification settings - Fork 16
feat(SF2.0/UpcomingDepartures): Show scheduled trips for bus and CR #2850
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
feat(SF2.0/UpcomingDepartures): Show scheduled trips for bus and CR #2850
Conversation
We will do this correctly later, so no need to maintain a special function for it now
thecristen
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One small thing to add now or later, otherwise nice!
| defp realtime_display(assigns) do | ||
| ~H""" | ||
| <span class="font-bold"> | ||
| <.icon type="icon-svg" name="icon-realtime-tracking" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's add an aria-label here!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call!
I'll treat that as a follow-up, since I'm not actually sure what aria-label to use (and because this isn't new - this code was moved, not added 🙂)
Features:
Upcoming Departureentries for bus and CR trips that are scheduled but don't have predictions.Asana Ticket: [SF/UD] Bus: Include predictions and scheduled-not-cancelled trips