Skip to content

Redesign 404 page #4154

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

Merged
merged 1 commit into from
Nov 15, 2021
Merged

Redesign 404 page #4154

merged 1 commit into from
Nov 15, 2021

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Nov 10, 2021

The logo comes from https://www.rustacean.net

Bildschirmfoto 2021-11-10 um 14 41 29

@Turbo87 Turbo87 added C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works A-frontend 🐹 labels Nov 10, 2021
@Turbo87 Turbo87 requested review from locks and pichfl November 10, 2021 20:11
@locks
Copy link
Contributor

locks commented Nov 15, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Nov 15, 2021

📌 Commit 79636de has been approved by locks

@bors
Copy link
Contributor

bors commented Nov 15, 2021

⌛ Testing commit 79636de with merge 7ed3f85...

@bors
Copy link
Contributor

bors commented Nov 15, 2021

☀️ Test successful - checks-actions
Approved by: locks
Pushing 7ed3f85 to master...

@bors bors merged commit 7ed3f85 into rust-lang:master Nov 15, 2021
@Turbo87 Turbo87 deleted the 404 branch November 15, 2021 10:19
bors added a commit that referenced this pull request Nov 16, 2021
Show 404 pages for unknown crates/versions

Instead of redirecting back to a known URL and showing a notification message, we now stay on the requested URL and show an error page instead.

<img width="721" alt="Bildschirmfoto 2021-11-10 um 14 41 29" src="https://user-images.githubusercontent.com/141300/141200350-d31a3b60-79a9-43eb-b104-aee1238d55f3.png">

(though with "Crate/Version not found" instead)

The implementation is a little questionable, but due to the way the `error` substates in Ember.js work I've found no other way to implement this.

This PR is based on #4154.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-frontend 🐹 C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants