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

Refactoring the 404 page #674

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

srishti666
Copy link

@srishti666 srishti666 commented Oct 14, 2022

πŸ‘¨β€πŸ’» Changes proposed

  • Changed the design of 404 page
  • Changed the page according to the theme of CodeIN website (yellow)
  • Added a button to the "Go to home?" option
  • Added hover effect to that button

βœ”οΈ Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

πŸ“„ Note to reviewers

πŸ“· Screenshots

image
image

closes #656

@netlify
Copy link

netlify bot commented Oct 14, 2022

βœ… Deploy Preview for codeincommunity ready!

Name Link
πŸ”¨ Latest commit f2042b9
πŸ” Latest deploy log https://app.netlify.com/sites/codeincommunity/deploys/63597b583ffcad00096af46d
😎 Deploy Preview https://deploy-preview-674--codeincommunity.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@error9098x
Copy link
Contributor

It's not responsive

@tamalCodes tamalCodes added bug Something isn't working Hacktoberfest-accepted 🚦 status : Requested changes Changes have been requested for these PRs labels Oct 14, 2022
Copy link
Collaborator

@tamalCodes tamalCodes left a comment

Choose a reason for hiding this comment

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

  • Can we get the issue number please ?

@srishti666
Copy link
Author

@tamalCodes
Copy link
Collaborator

Can you please provide the

  • Issue number in the PR in closes #issue_number_here format
  • Can you please resolve the conflicts and tag me ?

@srishti666 srishti666 changed the title Changed the UI of 404 page closes #674 Oct 26, 2022
@srishti666
Copy link
Author

@iamtamal I have resolved all the conflicts and provided the issue number in the PR.

@tamalCodes
Copy link
Collaborator

@iamtamal I have resolved all the conflicts and provided the issue number in the PR.

The thing is you have not even made an issue.
Else it would have been shown here :

image

background-image: url(https://cdn.dribbble.com/users/285475/screenshots/2083086/dribbble_1.gif);
height: 400px;
background-position: center;
body{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Donot make changes to body, make changes only to classnames. or id

border:2px solid rgb(247,184,1);
}
@media only screen and (max-width:700px){
body{
Copy link
Collaborator

Choose a reason for hiding this comment

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

Donot make changes to body , as this might affect other pages later on !

@tamalCodes tamalCodes changed the title closes #674 Refactoring the 404 page Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Hacktoberfest-accepted 🚦 status : Requested changes Changes have been requested for these PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Linkdin id link is broken
3 participants