-
Notifications
You must be signed in to change notification settings - Fork 7
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: Update route ladder dropdown #2802
Conversation
Co-authored-by: Josh Larson <jlarson@mbta.com>
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.
Mostly looks good! I made a few comments about CSS that I'd like to talk about, but don't necessarily need to block approval, and one visual issue that I think would be good fix before merging.
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.
Approving because this looks great, and the out-standing CSS questions require a team-wide consensus that doesn't exist yet. Not that they shouldn't be addressed, but I'm not ready to withhold my approval for this feature on it.
Asana ticket: https://app.asana.com/0/1205385723132845/1208333281632131
Pushing up my current work. I'm again confused by tests; to be fixed when I get back from time off