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

Various tailwind fixes #91

Closed
wants to merge 4 commits into from
Closed

Various tailwind fixes #91

wants to merge 4 commits into from

Conversation

robertmcabee
Copy link
Contributor

This is intended to improve the default index.tsx page.

  • Removes NextJS <Link> component
  • Whole card is clickable
  • Hover on card changes button styles
  • Hover animation duration reduced from 500ms to 300ms
  • changed parent <section> to <main> for better semantics
  • Card heights are consistent on each row

This is my first contribution to the project, hopefully everything looks okay!

nexxeln and others added 3 commits July 3, 2022 19:03
add custom publish script to build before publishing
<a target="_blank">Learn More</a>
</Link>
</button>
<section className="h-full transform group border-2 border-neutral-800 rounded-2xl duration-300 hover:scale-105 hover:border-blue-600 hover:-translate-y-1 hover:shadow-2xl">
Copy link
Member

Choose a reason for hiding this comment

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

Is hover:-translate-y correct? The extra - at the beginning in particular

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe so - its just a negative value in tailwind, i.e. negative margin on hover would be hover:-mt-8. It's from the initial tailwind styles, not my code so I'm not 100% confident.

@nexxeln
Copy link
Member

nexxeln commented Jul 4, 2022

We have some merge conflicts. Take a look at #89

@robertmcabee
Copy link
Contributor Author

Closing per nexxel to hopeful making merging easier. Will submit a new PR.

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.

3 participants