Skip to content

Conversation

@RAJVEER42
Copy link
Contributor

What GitHub issue is this PR for, if any?
Resolves #6440

What changed, and why?
This PR introduces subtle but effective UI enhancements to improve interactivity, visual feedback, and trustworthiness for users — especially first-time visitors and potential contributors.


✅ Changes made:
🔶 "Get In Touch" Button:

-> Added hover effects: background color transition, upward shift (-translate-y-0.5), and horizontal scale (scale-x-110)

-> Added will-change-transform and smooth transitions to improve responsiveness

🔶 Navbar Links (About, Testimonials, Contact):

-> Added hover:underline, hover:decoration-blue-300, and hover:opacity-90

-> Improves clarity, consistency, and feedback on navigation

🔶 Footer Links:

-> Updated "Ruby for Good" and "Storyset" links:

-> Added hover:underline and transition duration-100

-> Enhances accessibility and user confidence

🔶 Footer Email Link ([email protected]):

-> Added hover:underline, hover:text-green-300, and transition duration-100

-> Indicates it's a trustworthy and interactive email link

These small enhancements improve the site's polish and usability without altering layout or backend logic.

🔷How is this tested? (please write rspec and jest tests!) 💖

-> Manually tested across major browsers and screen sizes

-> Verified all hover effects work consistently and as intended

-> No layout regressions or accessibility issues found

Since this is a CSS-only change, no RSpec or Jest tests are required


Screenshots please :)
🔹 Navbar Hover Links

image

🔹 Footer Email & Links

image
image
image

🔹 "Get in Touch" Button

image


(just for a smile 😄)
Warning: May cause uncontrollable grinning and instant approval 😎💚

Small tweak, huge effect
Small polish, big impact! ✨

Cool dev
"Fixed a footer link and walked away like this."

~Thank you for your time and consideration! 😊
Excited to contribute and always open to feedback.

@RAJVEER42
Copy link
Contributor Author

Hi @compwron , @FireLemons , and @elasticspoon 👋

I’ve submitted a small UI enhancement PR that improves hover interactions for better user experience (Resolves #6440).
Whenever you get a chance, I’d really appreciate your review and feedback 🙌

Thank you! 😊

@compwron compwron requested a review from Copilot July 8, 2025 15:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Enhance UI interactivity and visual feedback by adding hover and transition effects to key elements.

  • Added hover underline, color, and opacity feedback to navbar links
  • Enhanced the “Get In Touch” button with transform-based hover transitions
  • Updated footer links and email link with underline, color change, and transition effects

Comment on lines 44 to 46
<a class="hidden md:block hover:underline hover:decoration-blue-300 hover:opacity-90" href="#about">About</a>
<a class="hidden md:block hover:underline hover:decoration-blue-300 hover:opacity-90" href="#testimonials">Testimonials</a>
<a class="hidden md:block hover:underline hover:decoration-blue-300 hover:opacity-90" href="#contact">Contact</a>
Copy link

Copilot AI Jul 8, 2025

Choose a reason for hiding this comment

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

[nitpick] The same hover and transition classes are repeated across navbar and footer links; consider extracting them into a shared utility or component to reduce duplication.

Suggested change
<a class="hidden md:block hover:underline hover:decoration-blue-300 hover:opacity-90" href="#about">About</a>
<a class="hidden md:block hover:underline hover:decoration-blue-300 hover:opacity-90" href="#testimonials">Testimonials</a>
<a class="hidden md:block hover:underline hover:decoration-blue-300 hover:opacity-90" href="#contact">Contact</a>
<a class="hidden md:block hover-link" href="#about">About</a>
<a class="hidden md:block hover-link" href="#testimonials">Testimonials</a>
<a class="hidden md:block hover-link" href="#contact">Contact</a>

Copilot uses AI. Check for mistakes.
@compwron
Copy link
Collaborator

I am not convinced that we want to merge this. Later I will look at the UI locally and see whether it seems more plausible than it looks from here.

@RAJVEER42
Copy link
Contributor Author

Sure @compwron,
Totally understand! Please take your time to review it locally — I’d love to hear your thoughts once you’ve had a chance to explore the UI in context. Let me know if there's anything specific you'd like me to improve or adjust in the meantime!

@RAJVEER42
Copy link
Contributor Author

Sure @compwron,
Just checking in — if you get a chance to look at it locally, I’d love to hear your thoughts! If there’s anything you’d like me to tweak, improve, or explain further, feel free to let me know.
Happy to make changes if needed — appreciate your time reviewing it! 😊

@compwron compwron merged commit acf9c61 into rubyforgood:main Jul 21, 2025
9 of 10 checks passed
@compwron
Copy link
Collaborator

I applied some fixes and will open a follow-up PR for any changes that seem needed.

@RAJVEER42
Copy link
Contributor Author

@compwron
Thanks for the quick fixes! That sounds great — looking forward to the follow-up PR. Let me know if you’d like any help reviewing or testing the upcoming changes.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants