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

Refine Pill Link design #1206

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

vandorsx
Copy link

@vandorsx vandorsx commented Aug 9, 2024

I had interacted with this component on the homepage and thought a subtle animation would be a simple but nice addition :)

I'm aware this is an unsolicited design PR, so feel free to close!

  • Adds a hover animation to the subtitle arrow
    • Custom, but based on Tailwind's default bounce animation
  • Adjusts opacity and positioning of the subtitle text decoration

Additionally, the unused Icon import was removed.

demo.mp4

Browser Test Checklist

I have tested this PR on at least three of the following browsers:

  • Chrome / Chromium
  • Firefox
  • Android Firefox
  • Safari
  • iOS Safari

Copy link

netlify bot commented Aug 9, 2024

‼️ Deploy request for astro-www-2 rejected.

Name Link
🔨 Latest commit 2c24610

On further thought, this one-off animation probably shouldn't be in Tailwind's config
@vandorsx vandorsx force-pushed the pilllink-animation branch from da27637 to de67f32 Compare August 9, 2024 15:42
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.

1 participant