Card component #40
Card component #40
Conversation
| @@ -0,0 +1,38 @@ | |||
| <div class="card-component card card-fit"> | |||
There was a problem hiding this comment.
instalar htmlbeautifier para darle formato automaticamente. Aqui estás usando 4 espacios de indentación. Deberían de ser 2
app/components/bali/card/index.js
Outdated
| // import { Controller } from '@hotwired/stimulus' | ||
| // | ||
| // export class CardController extends Controller { | ||
| // connect() { | ||
| // } | ||
| // | ||
| // disconnect() { | ||
| // } | ||
| // } |
| <%# post.tags.each do |tag| %> | ||
| <%#= link_to tag_path(tag) do %> | ||
| <!--<span class="tag is-info is-light"><%#= tag.name %></span>--> | ||
| <%# end %> | ||
| <%# end #%> |
There was a problem hiding this comment.
Elminar este código que ya no estamos usando aquí
app/components/bali/card/index.scss
Outdated
| @@ -0,0 +1,18 @@ | |||
| .column:not(.is-full) { | |||
There was a problem hiding this comment.
todo el CSS de un componente debe estar scoped bajo una clase con el nombre del componente. Ej. card-component.
Esto para que el CSS de cada componente solo impacte a los elementos del componente y no tenga impacto en otros elementos.
There was a problem hiding this comment.
Está bien si lo dejo como .card-component.card desde la línea 2? Tengo que evaluar si viene de una columna superior y no es "is-full", para que no le de estas alturas fijas al leader post
app/components/bali/card/index.scss
Outdated
| @@ -0,0 +1,18 @@ | |||
| .column:not(.is-full) { | |||
There was a problem hiding this comment.
Acostumbramos que el css este dentro de la clase nombre-del-componente-component, para que solo aplique los estilos al componente. Entonces en este caso sería
| .column:not(.is-full) { | |
| .card-component { | |
| ## CSS del componente | |
| } |
There was a problem hiding this comment.
Y sí está bien que lo ponga en esa línea aunque arriba tenga ".column:not(.is-full)"?
Co-authored-by: Miguel Frias <[email protected]>
|
Code Climate has analyzed commit 4f461cf and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 76.7% (1.9% change). View more on Code Climate. |
Agregar componente
card