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

Feature/infinite moving cards #61

Conversation

sivareddyuppathi
Copy link
Contributor

No description provided.

Copy link
Member

@rahul-vashishtha rahul-vashishtha left a comment

Choose a reason for hiding this comment

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

Hi @sivareddyuppathi,
Thanks for providing this component.

After reviewing, I found that this component seems like another implementation of Marquee component we already have. It has the same functionality and features.

https://inspira-ui.com/components/marquee

I think there is no need to have 2 components which are exactly similar in features & functionality.

@@ -0,0 +1,32 @@
---
title: Infinite Moving Cards
description: A customizable group of cards that move infinitely in a loop. Made with Framer Motion and Tailwind CSS.
Copy link
Member

Choose a reason for hiding this comment

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

We are not using Framer Motion.

@@ -50,7 +50,7 @@ export default withNuxt(
"import/newline-after-import": "error",
"import/no-duplicates": "error",
"import/no-unresolved": "off", //Need eslint-import-resolver-typescript, waiting for flatconfig and error fix on package side
"func-style": ["error", "declaration"],
"func-style": ["error", "declaration", { allowArrowFunctions: true }],
Copy link
Member

Choose a reason for hiding this comment

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

Arrow functions are not allowed.

@sivareddyuppathi
Copy link
Contributor Author

Yeah you're right @rahul-vashishtha I'm closing this 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.

2 participants