@EgnodiaSubmitted 8 days ago
leo-hailey
@leo-haileyAll comments
- @leo-haileyPosted 8 days ago
This looks awesome! The code is laid out very well! My only suggestion would be to add a bit more margin around the entire grid section to give it some space from the edge of the screen. Great job!
0 - @TarestaSubmitted 9 days agoWhat are you most proud of, and what would you do differently next time?
Implementing the grid layout was challenging, but I somehow figured it out. I will try to continue learning responsive design and how to make my code simpler.
What challenges did you encounter, and how did you overcome them?I had a hard time deciding whether to choose flexbox or grid. I understand that a two-dimensional layout grid is a good choice, but flexbox has wrapping options. I tried to build the page with both approaches and found it was easier with grid, so that got me settled.
What specific areas of your project would you like help with?I would like to further improve my understanding of responsive layouts and responsive design.
@leo-haileyPosted 9 days agoThis looks great! The code is easy to read, and it matches the design spec very well. I think the only suggestion I would have for you would be to just add a max-width to the paragraph containing "Our Artificial Intelligence...". Good Job!!!
1 - @jonathanudehSubmitted 23 days agoWhat challenges did you encounter, and how did you overcome them?
No challenge really. I just learnt more from practicing with grid
What specific areas of your project would you like help with?My projects are actually to big or too small, and thats because i dont know the measurements for the width and height that i am supposed to use. Hence I just do what looks good to me but while still following the design.
@leo-haileyPosted 19 days agoThis looks great! I would suggest adding a little bit of line-height to the .product-details to make it a little easier to read.
0
