Skip to content

improved animation #118

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

Closed
wants to merge 0 commits into from
Closed

Conversation

priyanshu1976
Copy link


name: Pull Request
about: Submit changes to the project for review and inclusion

Description

Enhanced the animation and hover interaction for the developer links section to improve user experience and interactivity.

Related Issue

This PR fixes #

Changes Made

  • Added smooth scale and bounce effects using Framer Motion.
  • On hover, animated background color transitions for the motion.div (content inside link).
  • Improved accessibility by ensuring better contrast (black background with white text/icon on hover).
  • Used spring animations for natural interaction feel.

Testing Performed

  • Tested hover interactions across Chrome, Firefox, and Edge.
  • Verified animations trigger smoothly with no layout shifts.
  • Confirmed text and icon visibility remains intact across states.
  • Checked responsiveness on different screen sizes.

Checklist

  • I have tested these changes locally and they work as expected.
  • I have added/updated tests that prove the effectiveness of these changes.
  • I have updated the documentation to reflect these changes, if applicable.
  • I have followed the project's coding style guidelines.
  • I have addressed the code review feedback from the previous submission, if applicable.

Additional Notes for Reviewers

  • The animation changes were made using Framer Motion and Tailwind CSS.
  • Let me know if you want the animation to be more subtle or interactive (e.g., ripple, glow, etc.).

Thank you for reviewing the PR! Looking forward to your feedback. 😊

@sa-fw-an
Copy link
Member

Please work only on the necessary tasks. I noticed that you have pushed unnecessary changes as well. Also make sure that the code builds and there is no Linting error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants