Skip to content

added 404 website themed page#102

Open
anushka11p wants to merge 3 commits into
codepvg:mainfrom
anushka11p:feat/custom-404-page
Open

added 404 website themed page#102
anushka11p wants to merge 3 commits into
codepvg:mainfrom
anushka11p:feat/custom-404-page

Conversation

@anushka11p
Copy link
Copy Markdown

Description

Adds a custom 404.html page 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

  • Added catch-all app.use() handler at the end of server.js that serves frontend/404.html with HTTP status 404
  • Created frontend/404.html with matrix rain background, green terminal palette, dynamic path injection via window.location.pathname, and navigation links back to /leaderboard and /registration

Type of Change

  • Bug fix
  • New feature
  • UI/Visual update
  • Documentation update
  • Refactor

Testing

  • Tested locally
  • Tested on mobile viewport (if applicable)
  • No console errors introduced

Checklist

  • My code follows the project's coding style
  • I have formatted my code locally using Prettier
  • I have performed a self-review of my code
  • My changes generate no new warnings or errors
  • I have updated documentation if required
  • I have linked the relevant issue

Screenshots / Screen Recording

Screenshot 2026-06-04 at 1 31 54 PM

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 4, 2026

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.

@jagdish-15
Copy link
Copy Markdown
Collaborator

/format

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 5, 2026

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:

  • Push an empty commit locally (git commit --allow-empty -m "Trigger builds" and push)
  • Close and immediately reopen this Pull Request.

@jagdish-15 jagdish-15 closed this Jun 5, 2026
@jagdish-15 jagdish-15 reopened this Jun 5, 2026
Copy link
Copy Markdown
Collaborator

@jagdish-15 jagdish-15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Add a custom 404 page

2 participants