-
Notifications
You must be signed in to change notification settings - Fork 2.6k
[Desktop] "404 Sorry, that page is missing." banner shows up when the server returns 'Internal Server Error' #8959
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
Comments
I guess we would need static image with number being a text. I like this idea as things would be lot less confusing if you go investigating what error code was returned. |
This new feature pops up even when some sites load correctly. See for example: https://matsim-vsp.github.io/covid-sim/v5 This is because many Github Pages sites use a common "trick" to redirect after a 404 page. For these sites, the Brave 404 error bar shows up even though the page loads correctly (from the user's perspective). This site behavior is possibly in error depending on your point of view, but it is common. Github Pages does not support server-side redirects, so SPA apps don't work with browser history. Many sites use a custom 404 page which immediately redirects to the SPA home page in order to get around this. Many pages online describe the hack, since Github Pages is a popular static site host nowadays: I am agnostic on whether this hack to get around Pages limitations is "correct", but Brave users are going to see a lot of error bars on sites that otherwise appear to be working. |
I like the idea, but the design of the banner is confusing. I've seeing the banner show up maybe 50 times in the last week and only today did I realised what it was – and that it was part of the browser and not something the webpages had rendered. It looks like a header on a webpage and not as something that is part of the browser UI. This is because it follows a familiar look with a "logo" in top-left corner and a bar across width of page, as seen below: I hope you'll consider updating the design, so it more closely resembles the browser UI. I suggest removing the image, removing the orange-purple colours, and placing the banner at the bottom of the page instead. A good example is the banner that shows up when you download a file, as seen below: |
@karenkliu Probably better if we make the image not say 404 to avoid confusion and make it a more archive-y/generic image. Trying to sub-brand this as the 404 service and including all the the error codes is confusing folks, especially web devs. Let's not do separate images per error code, e.g. a 502, 503 image. |
cc: @simonhong |
Designs added! |
Verification passed on
Verification passed on
Verification passed on
|
Description
A minority of users are confused by the 404 graphic displayed in the Wayback Machine banner because it doesn't match up to the actual error code (410, 502, etc.). Replace the graphic with a generic one that works for all error codes.
Original issue described:
Designs
For all states, replace the '404' image with the new image. Update body text to Poppins and remove other icons from the body copy.
After the user clicks to check if a saved version is available, show this state if there's no saved version:
Edge case
The Wayback Machine shows a fallback full-page error if the webpage doesn't have its own error page
Dark theme
Dark theme is supported:
Figma: https://www.figma.com/file/rz5shvMAAzqjWkfCN8dmAE/Desktop-404-Wayback-Machine?node-id=1%3A1805
The text was updated successfully, but these errors were encountered: