Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Targeta presen music

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Maqueda solo en HTML5 Y CSS3 usando de ayuda flex

Community feedback

Kimue 200

@dratinixgithub

Posted

-Bien, muy buen trabajo. De todos modos, te puedo dar dos pequeños consejos visuales para cambiar el 99% de tu trabajo por un 100%

Echa un vistazo a esto en el archivo de estilo:

-Incluir esta única propiedad en .target .bnt-payment

cursor: pointer;

-AGREGAR propiedades Hover y Active al botón de pago con estas 2 propiedades completas:

.target .bnt-payment:hover { transform: translate(-1px, -1px); }

.target .bnt-payment:active { transform: translate(1px, 1px); }

Como último consejo, ya de forma 'personal' me gustaría recomendar cambiar el @media de 1440px a 1140px. En mi caso especifico, no pude percibir nada del 1440 a full window si no fue por herramienta de desarrollador. (Claro, debido a que mi anchura no llega hasta allá) Me parece algo mas 'normal user friendly' usar dicha resolución

Marked as helpful

0

@IIpapuII

Posted

@dratinixgithub Buenas amigo, listo acatando las modificaciones tuyas hare el cambio para mejorar la estetica del mismo y añadire los @media para tres tamaños.

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord