@drigo101
Latest solutions
Latest comments
- P@joaofsds
You could have created one more breakpoint as the design specified, improve the styling of the share button and avoid the content to cut when clicked.
- P@KrisvHeijWhat are you most proud of, and what would you do differently next time?
I think my solution looks a lot like the design provided. I'm particularly proud of the use of flexbox for the cards. What i could do different next is that I perhaps would replace flexbox with grid.
What challenges did you encounter, and how did you overcome them?I had the most trouble getting the size of cards the same using flexbox. I had to look some things up using MDN Docs. I also had trouble with the color of the body background. The grey and white colors provided didn't seem to match the background color in the design. I decided to go for white because this was the color that came closest.
What specific areas of your project would you like help with?I would like some tips on my code (HTML & CSS). Maybe there are things I could have done different or code that I could have written shorter.
P@joaofsdsWell done, but you missed one of the breakpoints specified. :)
- @Zulkifli70What are you most proud of, and what would you do differently next time?
its kinda hard at first to change the image, for the mobile and desktop display, and now my I kinda improved to understand how media query works.
What challenges did you encounter, and how did you overcome them?To change the image from desktop to mobile display
What specific areas of your project would you like help with?Maybe how I make the media query for change the image, if you have a better solutions just tell me guys, I really appreciate it 🙏
P@joaofsdsHi Dev, very well done! Here some thoughts about your code:
About the image topic, you also could do it with the tag <picture> which selects different images depending on the breakpoint or use the image as background of a <div> and changing the background on the media query, the second one was my approach.
About the HTML, minding semantics, I think the "PERFUME" would be more suitable to use tags <small> or <span> for it and I also recommend you avoid using more than one <h1> per page.
Keep coding! o/
Marked as helpful - @REAPERali00What are you most proud of, and what would you do differently next time?
I managed to make thing quite organized, and got hang of them quite quickly
What challenges did you encounter, and how did you overcome them?there were some challenges with the lines separating the rows, which lead me to learn of border collapse property
What specific areas of your project would you like help with?Nothing much I suppose
P@joaofsdsvery good!
- @ayomidecoleWhat are you most proud of, and what would you do differently next time?
I went through it mostly by myself before I used the AI to fix some of the outlines
What challenges did you encounter, and how did you overcome them?Forgetting syntax for the most part
What specific areas of your project would you like help with?Just to continue practicing and getting more used to the syntax
P@joaofsdshey, good job.
try to include the responsiveness for mobile as well, I tried 440px and it worked fine.
- @byt3BusterP@joaofsds
Hey, good job :)
Some tips:
- You can remove the body to avoid the scrollbar.
- You can use css custom properties (variables) so you don't need to repeat yourself in color, spacing, etc.
