fix: Removed Framer Motion from Simple Toggle

epoll31 commented 4 months ago

Description

I removed the Framer Motion dependency from the Simple Toggle Component

Related Issue

Fixes #200

Proposed Changes

modified src/showcase/components/toggle/SimpleToggle.tsx
- removed framer motion references
- added local version of the cn function to the file so that it is visible in the code preview
- changed from arbitrary pixel sizes to using build in tailwind sizes (width, height, and padding)
- implemented same transition using raw tailwind instead of tailwind + framer motion
modified src/app/(docs)/components/toggle/simple-toggle/page.mdx
- removed usingFramer tag from docs page

Screenshots

If applicable, please include screenshots or GIFs to demonstrate the changes made.

Checklist

Please check the boxes that apply:

[x] I have rebased my branch on top of the latest main branch.
[x] I have tested the changes locally
[x] I ran npm run build and build is successful
[x] I have added necessary documentation (if applicable)
[x] I have updated the credits.md file (if applicable)

vercel[bot] commented 4 months ago

@epoll31 is attempting to deploy a commit to the SyntaxUI Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
syntax-ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 22, 2024 10:36am

Ansub commented 4 months ago

can we use something like this instead of adding a cn helper function?

function classNames(...classes) {
  return classes.filter(Boolean).join(' ')
}

you can use it like this:

className={classNames(
                tier.mostPopular ? 'lg:z-10 lg:rounded-b-none' : 'lg:mt-8',
                tierIdx === tiers.length - 1 ? 'lg:rounded-l-none' : '',
                'flex flex-col justify-between rounded-3xl bg-white p-8 ring-1 ring-gray-200 xl:p-10'
              )}

epoll31 commented 4 months ago

can we use something like this instead of adding a cn helper function?

Done :)

SyntaxUI / syntaxui