search

Aikodev-labs / Ghibli-API-platform

https://ghibli-api-platform.vercel.app
5 stars 0 forks source link

GA-26 componente boton creado #51

Closed DesSussu closed 1 year ago

DesSussu commented 1 year ago

Hola chicos!!

Componente Button creado con diferentes tamaños y tipos.

!! he intentado implementarlos a modo de prueba y no puedo moverlos de posicion. En el preview no están implementados

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated (UTC)
ghibli-api-platform ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 14, 2023 7:03pm
RosselloAgustin commented 1 year ago

La lógica se la ve bien. Lo que si, me hace ruido la parte de los estilos. Es correcto hacerla de ese modo? Si no me equivoco en algun momento cuando hay que usar cierto tipo de logica con Tailwind usamos `` (backtics) PERO no se si aplica para este caso

DesSussu commented 1 year ago

Reference in new iss

Hola Agus!!

Gracias por tu comentario! pero no estoy segura de entender completamente a qué te refieres. ¿Podrías explicármelo con más detalle para que pueda implementarlo y mejorar el código?

Quizás esté equivocada pero lo que sé del uso de los backticks es que generalmente se utilizan en JavaScript para crear plantillas de cadenas de texto donde se pueden incluir expresiones dentro de una cadena. En este código uso las props del componente en la expresión de un switch para compararlo y no he visto una necesidad inmediata de utilizarlos allí. Sin embargo, es posible que haya algún contexto específico en el código donde pueda aplicar backstick . Me podrías proporcionar más detalles o el fragmento de código donde crees que puedo usarlos?

Perdona tanto texto pero es más dificil explicarse por aquí jaja pero agradezco tu colaboración y estaré encantada de mejorar el códido con tu recomendaciones!!!

RosselloAgustin commented 1 year ago

Quizás no aplica lo que yo comentaba ya que como estás usando un switch (y trabajando por props) puede que no haga falta backtics. Sinceramente nunca usé switch para estos casos. Usaba un simple condicional (como en el navbar) y de ahí le aplicaba el estilo que requería.

Simplemente se me ocurrió esa opción en caso de que no te funcione el componente, pero ahora que miro detenidamente quizás no sea relevante. Perdón! Jaja

DesSussu commented 1 year ago

Acabo de aplicar los cambios que Josue me comento, decidme que os parece :)