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

Changes committed #470

Closed
wants to merge 3 commits into from
Closed

Conversation

Shivam250702
Copy link

@Shivam250702 Shivam250702 commented May 23, 2024

Related Issue

[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
#213

Description

[Please include a brief description of the changes or features added]
Added the Dark mode feature on Navbar

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Screenshots / videos (if applicable)

[Attach any relevant screenshots or videos demonstrating the changes]
image

Checklist:

  • I have performed a self-review of my code
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Additional context:

[Include any additional information or context that might be helpful for reviewers.]

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to Our repository.🎊 Thank you so much for taking the time to point this out.

@Suchitra-Sahoo
Copy link
Owner

Keep the website name AgriLearnNetwork near the logo only as it is now. Also kindly attach a video to demonstrate the changes both in dark and light mode. Also can you change the symbol of the modes like this:
image

@Suchitra-Sahoo
Copy link
Owner

@Shivam250702 are you working?

@Shivam250702
Copy link
Author

I have pushed the changes you required kindly check

@Suchitra-Sahoo
Copy link
Owner

Resolve the conflicts. Pull all changes from origin and add a video or screenshot to demonstrate how the website works both in dark and light mode.

@Shivam250702
Copy link
Author

image
image
image
image

@Suchitra-Sahoo
Copy link
Owner

Resolve the conflicts. Pull all recent changes from origin

@Suchitra-Sahoo
Copy link
Owner

In light mode you can keep some light shade of green. Make sure all the content is visible clearly whatever colour you choose.

@Suchitra-Sahoo
Copy link
Owner

Also check the placement of the mode option in small devices

@Suchitra-Sahoo
Copy link
Owner

Once you are done raise anothe PR

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.

None yet

2 participants