Skip to content

Conversation

@SHASHI9705
Copy link

@SHASHI9705 SHASHI9705 commented Nov 10, 2025

What kind of change does this PR introduce?

UI enhancement for better consistency between light and dark themes.


Issue Number:
Fixes #1889


Summary of Changes:

  • Improved the dark theme appearance for navigation tabs.
  • Added color highlight on hover (similar to light theme).
  • Updated selected tab styling to use a filled color instead of only an underline.
  • Ensured visual consistency and better accessibility in both light and dark modes.

Checklist:

  • My code follows the project’s contribution guidelines.
  • I’ve linked the relevant issue.
  • I’ve tested the changes locally.
  • The UI looks visually consistent across themes.

@SHASHI9705 SHASHI9705 requested a review from a team as a code owner November 10, 2025 10:26
@github-project-automation github-project-automation bot moved this to Ready to review in PR - Triage Group Nov 10, 2025
@github-actions
Copy link

Hi @SHASHI9705! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

@SHASHI9705
Copy link
Author

I've made the requested UI improvements for the dark/light theme consistency.
Everything has been tested and verified locally — please review and merge this PR when possible. 🙌

@github-actions
Copy link

github-actions bot commented Nov 10, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
website ✅ Ready (View Log) Visit Preview 47403a4

@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (1ca4a48) to head (47403a4).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #1900   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           30        30           
  Lines          633       633           
  Branches       196       196           
=========================================
  Hits           633       633           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@Utkarsh-123github Utkarsh-123github left a comment

Choose a reason for hiding this comment

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

Hi @SHASHI9705 , the main change looks good.
Could you please revert the modifications made to the package.json and yarn.lock files?
Thank you so much for your efforts 🚀

@github-project-automation github-project-automation bot moved this from Ready to review to Changes requested in PR - Triage Group Nov 11, 2025
@SHASHI9705
Copy link
Author

SHASHI9705 commented Nov 11, 2025

Hi @SHASHI9705 , the main change looks good. Could you please revert the modifications made to the package.json and yarn.lock files? Thank you so much for your efforts 🚀

I have made the changes of that files as previous one, please review and merge this PR when possible. 🙌 @Utkarsh-123github

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Changes requested

Development

Successfully merging this pull request may close these issues.

🌙 UI Enhancement: Improve active and hover states for navigation links in dark theme.

2 participants