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

feat: Implement smooth animation for Button #17

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

lazaroysr96
Copy link
Collaborator

@lazaroysr96 lazaroysr96 commented Dec 12, 2024

Implementación de animación de transición para el Button

  • Hacer que el ancho de carga entre el estado de carga y el estado predeterminado cambie suavemente.

Copy link

netlify bot commented Dec 12, 2024

Deploy Preview for erc20-vault failed.

Name Link
🔨 Latest commit 86f2ed4
🔍 Latest deploy log https://app.netlify.com/sites/erc20-vault/deploys/67674142bcd57800089d8e41

@lazaroysr96 lazaroysr96 self-assigned this Dec 12, 2024
@criss8X criss8X self-requested a review December 12, 2024 13:25
@criss8X criss8X changed the title feat: update button - implement animation feat: Implement smooth animation for Button Dec 12, 2024
Copy link
Collaborator

@criss8X criss8X left a comment

Choose a reason for hiding this comment

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

El animate-pulse no se esta ejecutando en el loading, hay que implementar mejoras en este boton, por ejemplo el estado de loading es igual que el de disabled osea se le baja demaciado la opacidad seria bueno que no fuera tan soso el color que mantuviera el color con mas opacidad que le de un color mas vivo pero que aun asi en esado loading no se pueda hacer click, osea mantener el funcionamiento del disabled con la UI de loading, no debe ser muy dificil de hacer

@lazaroysr96 lazaroysr96 requested a review from criss8X December 15, 2024 17:14
src/components/Button.tsx Outdated Show resolved Hide resolved
@lazaroysr96 lazaroysr96 requested a review from criss8X December 21, 2024 22:47
Copy link
Collaborator

@criss8X criss8X left a comment

Choose a reason for hiding this comment

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

Tienes que revisar ya que siges con el error de DOM, sigues teniendo un motion.button y dentro otro boton ya que tienes que tener e cuenta que el componente Tag es un button.

image

Un boton no puede ir dentro de otro boton, pusiste un div arriba pero dentro de ese div tienes un boton y dentro de ese boton otro boton. Trata de encontrar el equilibrio para las animaciones, ya que framermotion no se limita a transacciones layout solamente, puedes hacer varias transicciones de distintas formas

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