Skip to content

Conversation

mayur1377
Copy link
Contributor

Description

  • 🛠 Improvement to an existing snippet
    added some smooth animations for the code snippets , using framer here
Screen.Recording.2025-01-01.at.1.59.49.PM.mov

ps : ignore the christman theme :p

also noticed that the drop down bar is written from scratch and has too many custom css styles + too lengthy? would you like to have any fixes / improvements in that by using some standard library?

Copy link

netlify bot commented Jan 1, 2025

Deploy Preview for quicksnip ready!

Name Link
🔨 Latest commit 1696e76
🔍 Latest deploy log https://app.netlify.com/sites/quicksnip/deploys/677636632f064f000861029b
😎 Deploy Preview https://deploy-preview-78--quicksnip.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Yugveer06
Copy link
Contributor

Great. Some suggestions:

  • Use the new motion package as it is now separate from framer.
  • Respect prefers-reduced-motion for accessibility purposes.
  • Implement exit animations

Copy link
Collaborator

@Mathys-Gasnier Mathys-Gasnier left a comment

Choose a reason for hiding this comment

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

Hey, tanks for the contribution, I really like the idea of smoothing transitions.
I left a quick comment about some build files that you should have not commited, appart from that it looks good.

Copy link
Collaborator

@saminjay saminjay left a comment

Choose a reason for hiding this comment

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

Going from less snippets to more snippets has a smooth transition, but going other way is not. Its still snappy like before.

@technoph1le technoph1le added enhancement New feature or request update needed Code needs to be updated. labels Jan 1, 2025
@mayur1377
Copy link
Contributor Author

Going from less snippets to more snippets has a smooth transition, but going other way is not. Its still snappy like before.

thanks for pointing it out , fixed it!

Copy link
Collaborator

@saminjay saminjay left a comment

Choose a reason for hiding this comment

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

Everything seems to work fine now.

Just fix all the ESLint errors introduced by you in the PR.
run npm run lint to check what those errors are.
There are many linter errors, so I think you might not have eslint configured in your editor. It would help you fix these errors.

Since you are installing a new package, we need a final confirmation from @dostonnabotov.

Copy link
Collaborator

@psychlone77 psychlone77 left a comment

Choose a reason for hiding this comment

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

Tested, and everything looks good. 👍

@psychlone77 psychlone77 merged commit fc097aa into quicksnip-dev:main Jan 2, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request update needed Code needs to be updated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants