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
update pegasus & dashboard 404 to new sad bee layout #25913
Conversation
The dashboard page still doesn't have the navbar, doesn't have the footer, and isn't localized... but it has the friendlier look. Considering this v1 and we can do the other stuff later. |
<div class="error-child"> | ||
<img class="error-image" src="/shared/images/sad-bee-avatar.png" alt="Page not found"/> | ||
<h1>404: Page Not Found</h1> | ||
<div class="error-message">The page you are looking for doesn't exist at this address. Check the web address and resubmit. Or, go back to our homepage.</div> |
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.
Oh, we don't loc the 404 on dashboard?
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.
Correct. This page wasn't loc'ed before, but at least now the text is pulled out of the image. And we're one step closer to loc'ing later.
border: 1px solid #CCC; | ||
border-right-color: #999; | ||
border-bottom-color: #999; | ||
button { |
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.
It would be great to load this from existing CSS instead of duplicating.
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 will look into this for v2.
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.
If this looks good on mobile, then seems good to me.
Spec
Pegasus 404
Dashboard 404