@R3ygoski
Posted
¡Hola Julián, felicidades por completar este desafío! Quedó casi idéntico al diseño propuesto, mis más sinceras felicitaciones.
Quisiera comenzar hablando de un pequeño detalle: cuando está en la pantalla móvil responsiva, tu imagen tiene el border-radius
incorrecto, permanece igual que en la versión de escritorio.
Sobre el título que está un poco más grande, puedes usar width: 80%;
, eso lo hará parecer más similar al diseño.
Ahora, en cuanto a la parte semántica, está bien, pero hay algunas partes que pueden mejorarse. Por ejemplo, esta <section class="card">
, no creo que fuera realmente necesaria, ¿pero la utilizaste por Bootstrap, verdad?
Otra cosa que puedes cambiar son los encabezados. Usaste dos encabezados, <h2>
y <h6>
, pero hay una jerarquía de encabezados que se debe seguir. Siempre debes comenzar con <h1>
y descender progresivamente, sin saltarte ningún otro encabezado.
Tienes un fragmento que está mal, aquí está:
<h6 class="d-inline ms-4 pt-lg-1 previousPrice"><del>
$169.99</h6>
</del>
O <del>
se inicia dentro do <h6>
e se fecha depois do <h6>
, o que está errado.
Y también está la parte del <embed>
, lo correcto a usar allí es un <img/>
mismo, porque el <embed>
se utiliza para contenidos externos, no internos, como por ejemplo multimedia.
Y nuevamente, felicidades por tu proyecto, sigue practicando y mejorando. Si tienes alguna duda, ya sabes, solo comenta abajo.