Skip to content
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

[Desktop] "404 Sorry, that page is missing." banner shows up when the server returns 'Internal Server Error' #8959

Closed
domdfcoding opened this issue Apr 1, 2020 · 9 comments · Fixed by brave/brave-core#5840
Assignees
Labels
design A design change, especially one which needs input from the design team feature/wayback machine OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/exclude

Comments

@domdfcoding
Copy link

domdfcoding commented Apr 1, 2020

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:

The (new?) "404 Sorry, that page is missing." banner, which shows up when the user navigates to a page that doesn't exist and the server returns HTTP Status Code 404 (Not Found), also shows up when the server returns code 500 (Internal Server Error).

I can see the logic in showing the user a prompt to see if the page is available in the Wayback Machine, but the image on the left of the banner that shows "404" is inconsistent with the actual status code returned by the server.

The banner also shows up for code 410 (Gone), 502 (Bad Gateway), 503 (Service Unavailable) and 504 (Gateway Timeout), which all make sense for showing the banner for, but the image in the banner is still inconsistent.

This inconsistency may be confusing to the user if the website displays the HTTP Status Code as part of their custom error page.

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.

image

After the user clicks to check if a saved version is available, show this state if there's no saved version:

image

Edge case

The Wayback Machine shows a fallback full-page error if the webpage doesn't have its own error page

image

image

Dark theme

Dark theme is supported:

image

image

Figma: https://www.figma.com/file/rz5shvMAAzqjWkfCN8dmAE/Desktop-404-Wayback-Machine?node-id=1%3A1805

@NejcZdovc NejcZdovc added the design A design change, especially one which needs input from the design team label Apr 1, 2020
@NejcZdovc
Copy link
Contributor

NejcZdovc commented Apr 1, 2020

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.

@rebron rebron added the priority/P4 Planned work. We expect to get to it "soon". label Apr 7, 2020
@billyc
Copy link

billyc commented Apr 9, 2020

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.

@jsekamane
Copy link

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:

Screenshot 2020-04-17 at 13 40 16

Screenshot 2020-04-17 at 13 41 38

Screenshot 2020-04-17 at 13 41 19

Screenshot 2020-04-17 at 13 40 53

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:

Screenshot 2020-04-17 at 14 00 54

@rebron
Copy link
Collaborator

rebron commented Apr 22, 2020

cc: @karenkliu @jonathansampson

@rebron
Copy link
Collaborator

rebron commented Apr 22, 2020

@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.

@rebron
Copy link
Collaborator

rebron commented Apr 22, 2020

cc: @simonhong

@radounet-fr
Copy link

similar result here with a page describing a RESTful API definition (that returns an HTTP 200 code):

image

@karenkliu karenkliu assigned rebron and unassigned karenkliu Jun 11, 2020
@karenkliu
Copy link

Designs added!

@simonhong simonhong self-assigned this Jun 15, 2020
@simonhong simonhong added this to the 1.12.x - Nightly milestone Jun 15, 2020
@rebron rebron removed their assignment Jun 17, 2020
@rebron rebron changed the title "404 Sorry, that page is missing." banner shows up when the server returns 'Internal Server Error' [Desktop] "404 Sorry, that page is missing." banner shows up when the server returns 'Internal Server Error' Jul 20, 2020
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jul 27, 2020

Verification passed on


Brave | 1.12.101 Chromium: 84.0.4147.89 (Official Build) dev (64-bit)
-- | --
Revision | 19abfe7bcba9318a0b2a6bc6634a67fc834aa592-refs/branch-heads/4147@{#852}
OS | Windows 10 OS Version 1903 (Build 18362.959)

Light mode:
image

Dark mode:
image

Private tab:
image

Tor tab:
image

Guest window:
image


Verification passed on

Brave 1.12.102 Chromium: 84.0.4147.89 (Official Build) dev (64-bit)
Revision 19abfe7bcba9318a0b2a6bc6634a67fc834aa592-refs/branch-heads/4147@{#852}
OS Ubuntu 18.04 LTS

Light mode:
image

Dark mode:
image

Private tab:
image

Tor tab:
image

Guest window:
image


Verification passed on

Brave | 1.12.108 Chromium: 84.0.4147.105 (Official Build) (64-bit)
-- | --
Revision | a6b12dfad6663f13a7e16e9a42a6a4975374096b-refs/branch-heads/4147@{#943}
OS | macOS Version 10.14.6 (Build 18G3020)

Light mode:
8959 - light normal window

Dark mode:
8959 - dark normal window

Private window:
8959 - private window

Tor window:
8959 - tor window

Guest window:
8959 - guest window

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design A design change, especially one which needs input from the design team feature/wayback machine OS/Desktop priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.