Skip to content

fix: resolve roadmap module hover visibility issue in Light Theme - #307

Merged
mugenkyou merged 1 commit into
mugenkyou:mainfrom
AradhyaRay05:fix/roadmap-light-theme-hover-issue
Jul 4, 2026
Merged

fix: resolve roadmap module hover visibility issue in Light Theme#307
mugenkyou merged 1 commit into
mugenkyou:mainfrom
AradhyaRay05:fix/roadmap-light-theme-hover-issue

Conversation

@AradhyaRay05

Copy link
Copy Markdown
Contributor

Pull Request Summary

Fixes #305

This PR resolves the Roadmap module hover visibility issue in Light Theme. Some roadmap module cards/items became visually unclear or appeared blank on hover due to styling and contrast inconsistencies. The update improves readability and ensures a consistent hover experience in Light Theme while preserving Dark Theme behavior.


Changes Introduced

  • Fixed hover visibility issue in Roadmap module cards/items for Light Theme
  • Updated styling in roadmap_assets/css/sub-roadmap.css to improve contrast and readability
  • Preserved existing Dark Theme behavior without regressions

Screenshots / Demo (for UI changes)

Before After
image image

Checklist

Please ensure the following before requesting review:

  • Code follows project conventions and best practices.
  • Feature or fix works correctly on both mobile and desktop.
  • No new console errors or accessibility regressions.
  • Documentation updated if applicable.
  • Visuals attached for UI-related updates.

Additional Notes

This fix is limited to hover styling improvements for Roadmap module cards/items in Light Theme and does not affect existing functionality or Dark Theme behavior.

@github-actions github-actions Bot added awaiting review bug Something isn't working nsoc26 labels Jun 15, 2026
@github-actions
github-actions Bot requested a review from mugenkyou June 15, 2026 16:48
@github-actions

Copy link
Copy Markdown

📥 Pull Request Received

Thank you for your contribution to College Daddy. Your pull request has been received and is currently under review.

🔗 Linked Issues

✅ Pre-Merge Checklist

Please ensure the following requirements are met:

  • Changes adhere to contribution guidelines
  • Related issues are referenced in description
  • All changes tested and verified locally
  • Code follows project standards

👤 Reviewer Assigned

@mugenkyou has been assigned to review this pull request.


Our team will review your submission shortly. We appreciate your effort in improving the platform for students.

@mugenkyou mugenkyou added the level-2 Intermediate issue; moderate JS or UI work, needs some project understanding. label Jul 4, 2026
@mugenkyou
mugenkyou merged commit bff3228 into mugenkyou:main Jul 4, 2026
1 of 2 checks passed
@github-actions

github-actions Bot commented Jul 4, 2026

Copy link
Copy Markdown

🎉 Pull Request Merged Successfully

Contributor: @AradhyaRay05

Your pull request has been successfully merged into the main codebase. Thank you for your valuable contribution to College Daddy.

🚀 Next Steps

We encourage you to:

  • ⭐ Star the repository to support the project
  • 👀 Watch for updates and participate in discussions
  • 🔄 Continue contributing to enhance the platform

We value your commitment to improving educational technology and look forward to your continued involvement.

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

Labels

bug Something isn't working level-2 Intermediate issue; moderate JS or UI work, needs some project understanding. nsoc26

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Roadmap module cards become blank on hover in Light Theme

2 participants