@boysersSubmitted 4 months ago
mangentemoncarlo
@mangentemoncarloAll comments
- @mangentemoncarloPosted 4 months ago
The solution is pixel-perfect. The codes are well-structured and very clean. It is readable and easy to follow.
0 - @EugeneVuongSubmitted 5 months agoWhat specific areas of your project would you like help with?
- Is my usage of px, rem, and em correct? I have only used rem and px throughout my code because I couldn't find a suitable place for em. If you have any suggestions on where I can use each of these units better, I would love to know.
- Is my responsiveness good and is everything up to scale depending on the font size and zoom of the browser? Any areas of improvement regarding responsiveness?
- Is there a better way to make the padding responsive depending on the screen size? I used
padding: clamp(1.5rem, 4vw, 2.4rem);
Not sure if that is the best method.
- @KingAndre13Submitted 4 months ago
- @alberto-rjSubmitted 8 months agoWhat specific areas of your project would you like help with?
👍 Any suggestions on how I can improve are welcome!
- @Bamo-D-AbdallahSubmitted 4 months agoWhat are you most proud of, and what would you do differently next time?
I used grid and could manage make my files some how a little organized.
What specific areas of your project would you like help with?The overall code quality and any suggestion would be appreciated.
@mangentemoncarloPosted 4 months agoThe layout looks good on different screen sizes.
0 - @samuel-aduSubmitted over 2 years ago@mangentemoncarloPosted 4 months ago
The solution looks great. It just need to be the same width between the image and the content.
0