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

fix dark mode bugs with border-width and ProfileCalendar #25

Merged
merged 4 commits into from
Oct 19, 2022

Conversation

xnatsuk
Copy link
Contributor

@xnatsuk xnatsuk commented Oct 13, 2022

Closes #24

@vercel
Copy link

vercel bot commented Oct 14, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @Mridul2820 on Vercel.

@Mridul2820 first needs to authorize it.

@xnatsuk xnatsuk changed the title Fix dark mode bugs with border-width and ProfileCalendar fix dark mode bugs with border-width and ProfileCalendar Oct 14, 2022
@vercel
Copy link

vercel bot commented Oct 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
git-to-get ✅ Ready (Inspect) Visit Preview Oct 19, 2022 at 2:28AM (UTC)

@Mridul2820
Copy link
Owner

Hi @xnatsuk, borders are required. Without the borders, the cards are separate from each other.

@xnatsuk
Copy link
Contributor Author

xnatsuk commented Oct 14, 2022

Seemed okay to me. I'll add them again

Removed dark only borders, making elements change size on theme change.
@xnatsuk
Copy link
Contributor Author

xnatsuk commented Oct 14, 2022

@Mridul2820 borders should be the same size across both themes now.

@Mridul2820
Copy link
Owner

Mridul2820 commented Oct 15, 2022

Hi @xnatsuk, a few things if you enhance for the dark mode, it will look great

  • user card URL color
  • User stats captions color
  • User followers name color
  • Most Forked, Most Stared, Contribution Graph axis label color

@xnatsuk
Copy link
Contributor Author

xnatsuk commented Oct 18, 2022

@Mridul2820 I didn't fix the axis label color because I'm not familiar with it's dependency, but the other colors are more visible now.

@Mridul2820 Mridul2820 merged commit c961828 into Mridul2820:main Oct 19, 2022
@Mridul2820
Copy link
Owner

Thank you for contributing! 🚀

@xnatsuk xnatsuk deleted the fix-darkMode branch October 22, 2022 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest Hacktoberfest hacktoberfest-accepted Hacktoberfest Accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bugs in dark theme
2 participants