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

update Rate-us and Log-out in navbar #309

Merged
merged 2 commits into from
Jun 15, 2024
Merged

update Rate-us and Log-out in navbar #309

merged 2 commits into from
Jun 15, 2024

Conversation

Varshi45
Copy link
Contributor

This pull request adds the functionality of dark mode to the Rate-us page. Additionally, this also change the color of the logout button in the navbar WRT on the mode (dark / light).

changes made:

  • removed the background image
  • changed the background color of the Rate-us form
  • the logout button now changes WRT mode (dark / light)

Fixes #289

Type of change

Please give a X on it which is applicable

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactor Code
  • A documentation update
  • Others(mentioned in the issue number)

How Has This Been Tested?

Manual trail and error is done to get the color pallet matching with existing pallet, in both dark and light modes.

Screenshorts and Vedios:

Screenshot 2024-06-11 211140
Screenshot 2024-06-11 211155

Checklist:

give a X on it which is applicable

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

Copy link

vercel bot commented Jun 12, 2024

@Varshi45 is attempting to deploy a commit to the vansh69's projects Team on Vercel.

A member of the Team first needs to authorize it.

@hustlerZzZ hustlerZzZ added good first issue Good for newcomers gssoc This label is attached to the issues related to Gssoc'24 level1 labels Jun 15, 2024
Copy link
Collaborator

@hustlerZzZ hustlerZzZ left a comment

Choose a reason for hiding this comment

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

Great work!

@hustlerZzZ hustlerZzZ merged commit a75003e into VanshKing30:main Jun 15, 2024
0 of 2 checks passed
Copy link

Congratulations, Your pull request has been successfully merged 🥳🎉 Thank you for your contribution to the project 🚀 Keep Contributing!! ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers gssoc This label is attached to the issues related to Gssoc'24 level1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: Dark Mode for Rate-us section
2 participants