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

light/dark mode toggle button added #244

Closed
wants to merge 1 commit into from

Conversation

aray1608
Copy link

@aray1608 aray1608 commented May 20, 2024

PR Description

I have added a light/dark mode toggle button. Although i had some issue with its placement, i hope it gets reviewed and any input is welcome.

Screenshot 2024-05-21 010507
Screenshot 2024-05-21 010518

Related Issues: Issue for which you are raising a PR for

The sight didnt have a light mode.

Issue

[Issue number or description]

Checklist

  • I have gone through the [contributing guide](## PR Description

Add your description

Related Issues: Issue for which you are raising a PR for

  • Closes #

Issue

[Issue number or description]

Checklist

  • I have gone through the contributing guide
  • I have updated my branch and synced it with project main branch before making this PR
  • Is this a bug fix/enhancement/documentation changes

Undertaking

I declare that:

  1. The content I am submitting is original and has not been plagiarized.
  2. No portion of the work has been copied from any other source without proper attribution.
  3. The work has been checked for plagiarism, and I assure its authenticity.
  • I have updated my branch and synced it with project main branch before making this PR
  • Is this a bug fix/enhancement/documentation changes
  • Part of GSSOC
  • Tested for any breaking changes
  • Other relevant checks completed

I understand that any violation of this undertaking may have legal consequences that I will bear and could result in the withdrawal of any recognition associated with the work.

  • I Agree

Copy link
Contributor

@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.

@Anishkagupta004
Copy link

The sign is overlapping over Doctor section

@varshith257
Copy link
Collaborator

@aray1608 How without any code changes made a PR?

@varshith257 varshith257 added invalid This doesn't seem right question Further information is requested labels May 21, 2024
@aray1608
Copy link
Author

@aray1608 How without any code changes made a PR?

I had made changes in the index.html file

@varshith257
Copy link
Collaborator

Where are the changes? I can't see any file changes in the PR

@aray1608
Copy link
Author

Where are the changes? I can't see any file changes in the PR

Sorry, just noticed that now. The pictures I have attached above reflect the changes I had made. I think I messed up while making the PR.

In the meanwhile I am learning stuff and was planning to make some other changes too in the code, I'll fix that and make a new PR. Please give me some time.

@varshith257
Copy link
Collaborator

@aray1608 Raise the another PR when it's done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants