Daniel Ayeni
@DannimationsAll comments
- @Tasin269Submitted 13 days ago@DannimationsPosted 13 days ago
You did a good job in inserting the quotes as the background image for the first card. You still need to make the background colors of those two cards white, and maybe consider re-sizing the whole thing. :)
0 - @velvet-jediSubmitted 13 days ago@DannimationsPosted 13 days ago
You did a really good job making it responsive with multiple screens, all you need to correct is the padding in the cards, but asides that, its really good.
Marked as helpful0 - @aurelienfliegerSubmitted 16 days agoWhat are you most proud of, and what would you do differently next time?
I successfully implemented the cart icon using the ::before pseudo-element in CSS - no HTML needed, since this is only a decorative icon, the screen readers do not need to see it.
What challenges did you encounter, and how did you overcome them?I was not sure whether to use clamp() or the traditional min-width & max-width for the width.
I experimented with clamp but did not find the results controlled enough. It provided a janky interface. I need to experiment more.
What specific areas of your project would you like help with?/
@DannimationsPosted 16 days agoIt's a bit bigger, but asides that you nailed it! Good job man.:)
0 - @JacksonRuddSubmitted 26 days agoWhat are you most proud of, and what would you do differently next time?
This was my first time using a table, or horizontal breaks. I also used a media query, but not sure if there is a better way to make this responsive.
What challenges did you encounter, and how did you overcome them?I had some trouble making this responsive. I'm still using some
min-width
and some media queries. I want to learn more about doing this the right way. - @FirstHalcyonSubmitted 29 days agoWhat are you most proud of, and what would you do differently next time?
Last two exercises were Flex-only, so I opted to do just Grid with this one. Managed to completely avoid using divs and got by with just semantic elements.
Really happy with how it all came together. Flex seemed like the obvious choice here and I haven't practiced Grid in a while, so it was a nice change of pace.
What challenges did you encounter, and how did you overcome them?Was a bit rusty on Grid, so it was a bit cumbersome at first. It was just a single column of elements, so Flex was much more fitting. Still, by the end, I got a good hang of it.
@DannimationsPosted 29 days agoYou did a very good job, especially with the dimensions of the main div, which are almost identical to that of the original project. Keep at it! :)
1 - @abedbortolSubmitted 29 days ago@DannimationsPosted 29 days ago
.....I think you may have pushed the wrong project, or there was an error in your pushing, because this hardly looks like the original project.
0 - @PREETHAM-HTSubmitted 29 days ago@DannimationsPosted 29 days ago
Good job, just put the main div at the center of the page. You can achieve this by making it's position relative, then use the top, left and right attributes to position it accordingly.
0