added 404 website themed page#102
Conversation
|
Thank you for submitting a pull request. Please ensure your changes comply with the project's contribution guidelines and that all workflow checks pass successfully. A maintainer will review your submission shortly. |
|
/format |
|
I have successfully run Prettier and pushed the formatting fixes to this PR. Note for Contributors: Because this commit was pushed by a bot, GitHub will not automatically re-run the CI checks. To trigger them to pass, you must either:
|
jagdish-15
left a comment
There was a problem hiding this comment.
I noticed the page still has responsiveness issues on smaller mobile screens, so I'd like to get that addressed before merging.
I've already pushed a small maintainer commit for some minor layout/styling refinements and also ran formatting on the PR, so please pull the latest changes from your branch before continuing.
Once the mobile responsiveness issues are resolved as well, this should be good to merge.
Description
Adds a custom
404.htmlpage served by the Express catch-all middleware for all unmatched routes. Previously, undefined routes (e.g./xyz) returned a blank Express fallback with no way back to the app. This fix serves a themed 404 page consistent with the site's terminal/matrix aesthetic.Linked Issue
Fixes #97
Changes Made
app.use()handler at the end ofserver.jsthat servesfrontend/404.htmlwith HTTP status 404frontend/404.htmlwith matrix rain background, green terminal palette, dynamic path injection viawindow.location.pathname, and navigation links back to/leaderboardand/registrationType of Change
Testing
Checklist
Screenshots / Screen Recording