XYshop• 100
@XYshop
Submitted
Vinncent Miller
@centralparkshark
All comments
- Vinncent Miller• 150
@centralparkshark
Posted
Everything looks great! The mobile version works well, hover and active states, and even form validation all work :) Awesome job
0 - samir-Deve• 350
@samir-Deve
Submitted
What are you most proud of, and what would you do differently next time?
...
What challenges did you encounter, and how did you overcome them?
I was facing terrible with design and positioning that rectangular pointing to share button but luckily I found out how to create one
What specific areas of your project would you like help with?
The project I think has still a little bit of bug I will fix that myself latter
Vinncent Miller• 150@centralparkshark
Posted
I think I found the bug.. When you click the share button on a small screen (mobile), the block disappears with the button so you can no longer see either. The rest looks pretty good though I would try to stick closer to the design files.
0 - PMohd Sadaf• 450
@MsadafK
Submitted
Vinncent Miller• 150@centralparkshark
Posted
Looks good to me!!
1 - AntonB12• 140
@AntonB12
Submitted
What are you most proud of, and what would you do differently next time?
estoy orgulloso de que a pesar de tener algunas complicaciones con este ejercicio puede sacarlo adelante solo con los conocimientos que he adquirido tanto realizando los ejercicios aqui y los conocimientos de mi carrera, solo algunos temas tuve que investigarlos sin embargo esto es parte siempre del conocimiento
What challenges did you encounter, and how did you overcome them?
principalmente el de como hacer que solo aparezca el borde de arriba de cada uno de los contenedores con los que se trabajo con este ejercicio, ademas de tambien en un principio la distribucion de los contenedores
What specific areas of your project would you like help with?
todos los comentarios de ayuda y mejora son bienvenidos, me gustaria mucho poder leer en que aspectos puedo mejorar mi ejercicio tanto a nivel html como css
Vinncent Miller• 150@centralparkshark
Posted
Your design looks great! For the top border, I would probably use {border-top: 5px solid color}
0 - feldspar58• 160
@feldspar58
Submitted
Vinncent Miller• 150@centralparkshark
Posted
Looks awesome! For border-radius you can set individual corners to have different values (border-radius: 8px 0 0 8px)
Marked as helpful
0 - Martin Malyska• 190
@scaphyxx
Submitted
Vinncent Miller• 150@centralparkshark
Posted
Spacing was difficult on this one but you did great!
0 - Kapkrazy• 190
@Kapkrazy
Submitted
What are you most proud of, and what would you do differently next time?
I had fun with this one.
What challenges did you encounter, and how did you overcome them?
Getting the links to stretch the width of the card was a head scratcher for a good hour.
Vinncent Miller• 150@centralparkshark
Posted
Your css covers all its bases. Looks good tho!
Marked as helpful
0 - Kripesh Raj panta• 40
@Mr-Kripesh
Submitted
Vinncent Miller• 150@centralparkshark
Posted
Looks great! Your classes could probably be named more descriptive but it works for a project this small
0 - ramez bakeer• 120
@ramezbakeer
Submitted
What are you most proud of, and what would you do differently next time?
Making my first project
I proud of making my first simple project, and I hope to be professional developer
What challenges did you encounter, and how did you overcome them?
The challenge was simple so it was easy
What specific areas of your project would you like help with?
I need a feedback on the CSS code
Vinncent Miller• 150@centralparkshark
Posted
Looks great! Could use the figma file if you want pixel perfect designs, but isn't really necessary
0
