@lisslealSubmitted over 1 year ago
Fabian Andrade
@aluiggi96All comments
- @aluiggi96Posted over 1 year ago
excelente trabajo.... solo te falto el cursor:pointer cuando el cursor pasa sobre el boton de add to cart... el resto aparte se ve bien
0 - @AlejandraaragonSubmitted over 1 year ago@aluiggi96Posted over 1 year ago
Muy bien hecho... solo un pequeno detalle las letras en negritas son un chin mas grandes que las de abajo... ademas el padding de la descripcion del qr es ligeramente mayor al padding de la tarjeta actual. De igual forma se ve muy bien :)
0 - @sue-limaSubmitted over 1 year ago@aluiggi96Posted over 1 year ago
excellent, but dont forget there is a lil margin top on mobile design... the rest 100 :)
0 - @projectfinalaudioSubmitted over 1 year ago
What can I do to make the mock up better and more efficient?
@aluiggi96Posted over 1 year agogood try!... I suggest you follow mine or see this video https://www.youtube.com/watch?v=KqFAs5d3Yl8 who made different way of mine but same result
0 - @aluiggi96Submitted over 1 year ago@aluiggi96Posted over 1 year ago
This design is made on 1440px W so it's totally reponsive and adjust to different sizes of screen... anyways share me your opinions. thanks >_<
1 - @a-costasSubmitted over 1 year ago
I had started out by building the desktop version first and trying to use media tags to adjust for mobile, but had to shift gears and change my approach to be mobile first. I also ran into a couple of problems caused by not having CSS reset, so that's something I'll be more mindful of in the future.
I'd love any feedback regarding how to write cleaner and more accessible code especially, but all feedback is appreciated!
- @GabrielCarrillo93Submitted over 1 year ago
HI! A new challenge today. I had this one saved for the longest time but here it is. Thanks in advance for your feedback
@aluiggi96Posted over 1 year agoGran trabajo... cosas a destacar son que la imagen pierde sus dimensiones al disminuir el navegador... podrias ayudarte con height: 100% y width en auto... y algo desimportante es el fondo beige. el resto excelente
1 - @lisslealSubmitted over 1 year ago
In this project I used an each for the first time, It was very useful for applying different colors at background and text of the components.
@aluiggi96Posted over 1 year agoPor el momento todo bien... Me falta ver tu código pero deberías agregar una pequeña transición al botón al cambiar de color para que no sea ta abrupto al cambio... :)
0
