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

[BOOK] - Elaboración del componente Button #3

Open
4 tasks
D3Portillo opened this issue Sep 20, 2020 · 6 comments · May be fixed by #4
Open
4 tasks

[BOOK] - Elaboración del componente Button #3

D3Portillo opened this issue Sep 20, 2020 · 6 comments · May be fixed by #4

Comments

@D3Portillo
Copy link
Member

D3Portillo commented Sep 20, 2020

Notes/Feedback:

Gracias por tu colaboración en para los componentes de UI en la comunidad.
Los siguientes botones debe ser fluid y adaptar su ancho al contenedor. Además se necesita de 2 "sabores" del componente, estás libre de decidir cómo puede funcionar la lógica para la selección del "sabor", ten en cuenta que es necesario sí tener un sabor por defecto del componente.

Que se espera pueda hacer el boton

  • Se podrá ajustar las clases pasandolas por la prop className
  • El nombre del componente es Button

Clases de tailwind en el componente

bg-yellow text-black font-black text-xl px-6 py-5

Recursos/Resources:

Todo

  • Elaboración del "sabor" Primary del boton
  • Elaboración del "sabor" Secundary del boton
  • Test local importando el componente realizado
  • Integrado a master

Información de TailwindCSS

Remember you can suggest a better way of doing this, just spill the beans : )

@D3Portillo D3Portillo self-assigned this Sep 20, 2020
@D3Portillo D3Portillo changed the title [BOOK] - Resolves tailwindcss related bug [BOOK] - Elaboración del componente Button Sep 20, 2020
@D3Portillo D3Portillo assigned D3Portillo and unassigned D3Portillo Sep 20, 2020
@D3Portillo
Copy link
Member Author

@X905 Gracias por colaborar en el desarrollo del componente, espero tu PR, tienes asignado el issue : )

@X905
Copy link

X905 commented Sep 21, 2020

👨‍💻 me alegra poder colaborar!

@D3Portillo
Copy link
Member Author

He movido la task a "in progress" , muchas gracias @X905 , espero tu PR :'3

@D3Portillo
Copy link
Member Author

Por cierto, para los :hover puedes usar las clases de tailwind shadow-yellow y shadow-grey

@X905
Copy link

X905 commented Sep 22, 2020

Hola he hecho un PR (#4 ) con el nuevo componente, me gustaría tener una crítica, no estoy seguro si esa manera es la correcta para los sabores, ya que al final he creado dos elementos (Button y ButtonGrey), se podría descartar ButtonGrey ya que Button puede recibir las clases para tener el segundo sabor.

@D3Portillo
Copy link
Member Author

Hola @X905 , he dejado unos comentarios de cómo podría mejorar un poquin el PR #4 . Muchas gracias ❤️

@D3Portillo D3Portillo linked a pull request Sep 23, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants