If you check my code, find any errors, and want to give your suggestions, I would be very grateful for the time and effort you put into helping me improve my code and sharing your knowledge. 😊
Gustavo Sanchez
@gusanchefullstackAll comments
- @lutfiismail52What specific areas of your project would you like help with?P@gusanchefullstack
Good job. You could estimate the pixels between elements with trial and error.
- @toxa-devP@gusanchefullstack
Great job!
- P@developer-rubenP@gusanchefullstack
Awesome!
- P@Matthieu83600P@gusanchefullstack
Nice job!
- @DiegoMy7P@gusanchefullstack
Good job. You could try to fix the font suggested and margin/padding px based in images provided in challenge to get more familiar with measurements in px.
- P@clickglueWhat are you most proud of, and what would you do differently next time?
Start to learn responsive layout and media queries. Structuring HTML and CSS becomes more and more routine. Maybe use figma file to stop guesswork?
What challenges did you encounter, and how did you overcome them?using em, rem and pixels. Watched video.
What specific areas of your project would you like help with?Comments on proper structuring, use of BEM maybe?
P@gusanchefullstackGood job. I suggest a margin top of container in desktop media querie for 175px.
Marked as helpful - @meghaspatil1What are you most proud of, and what would you do differently next time?
The proud thing is that I managed the frame and 'div' it into sections to make perfect divand labels for them.
What challenges did you encounter, and how did you overcome them?This time, the actual challenge I faced was making it responsive and set to media series.
What specific areas of your project would you like help with?make me more understand about media queries.
P@gusanchefullstackGreat job! Just considering width of card
- @ayoubms8P@gusanchefullstack
You could remove the attribution code to fix perfectly vs design in figma. Good job
- @KitalpharWhat challenges did you encounter, and how did you overcome them?
I was aiming to write it with data injection in mind, but i think i may have just ended up overcomplicating for no reason.
P@gusanchefullstackAwesome! Great job.
- @FasihnasirP@gusanchefullstack
Good job. Some details: Border radius for card should be 20px Try to use image width and height to 288px.