-
Notifications
You must be signed in to change notification settings - Fork 2
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
805 make dashboard links clickable #1326
Conversation
…o 805-make-dashboard-links-clickable
…he Dashboard Landing screen. A better way to view all statuses will come with a Dashboard redesign.
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.
LGTM
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.
I like the use of an actual button!
One styling observation: It seems like the font size was carried over from the original version with the blue box. Now, without the blue box, the font appears significantly larger in comparison to the rest. I'm curious if this was intentional. If it was, just disregard this comment. If not, I think you could bring it down .5rem or so to help it look more balanced
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.
✅
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.
Thanks, @jonathanbobel!
Description
The dashboard had a large clickable area that didn't look like an actual link or button that allowed the user to drill down into a Status page for their recent messages. This fix removed those areas and created one single "Details" button that is more self-explanatory and allows for the same functionality.
Additionally, I removed the section that showed failures (and the subsequent confusing link to just the failures section). The user can still click on the Details button and see the combined status page, including failures.
Upcoming designs will have data visualizations to better show the Status - all current Delivered/Pending/Failed.