ICode88• 280
@ICode88
Submitted
Yanna
@yannancls
All comments
- Yanna• 120
@yannancls
Posted
The website was responsive and good, however my recommendation is to add a max-width to the main tag so that it looks good even on a large screen. Also, instead of using the div element to add the background picture in CSS, using an img tag would be preferable because it would allow you to apply CSS easily and shorten your code. Finally, I noticed that when I decreased the width, the gaps for card 5 were inconsistent, which I believe you can improve much more. But overall, it was excellent. Great job!
Marked as helpful
0 - Fahim-Haowlader• 100
@Fahim-Haowlader
Submitted
Yanna• 120@yannancls
Posted
Reading your CSS file confused me. It doesn't appear to be the correct file or anything, and I think you utilize Tailwind which is cool. And because I haven't tried to learn it yet, it makes it challenging for me to check your CSS. Moreover, the design of your webpage does not exactly portray the intended result based on the photo. I believe you did not use the font family that was provided to us as well as the colors; font sizes can be modified. Also, it is not responsive. It would be great to make it one to improve it.
0 - Emerson Silva• 30
@emersonescodes
Submitted
Yanna• 120@yannancls
Posted
After reviewing your work, I saw that the HTML file was great and the website is responsive. However, some fonts are too huge or small, so you can change them to make your work better. Furthermore, the hover button for the "Add to Cart" was not implemented. Lastly, I appreciate how you have divided your CSS files into distinct groups, though there was a duplicate "* { }". But overall, excellent work!
0 - Ayoub• 40
@ayoubms8
Submitted
Yanna• 120@yannancls
Posted
The h1, h2, and h3 tags' font sizes could have been larger. Examining the image of the designed result, I believe the <li> inside the <ul> are brown. The padding between the image and h1 may be reduced, and the appearance of the table at the end can be improved by making it centered or justifying it. Finally, decide if you want to name classes in Pascal, Camel case, or another format. Overall, it is responsive, you did a wonderful job!
0 @fernandatollotti
Submitted
What are you most proud of, and what would you do differently next time?
In this project, I take pride in successfully implementing CSS techniques to enhance my knowledge and skills in frontend development.
What challenges did you encounter, and how did you overcome them?
I did not encounter any difficulty with the challenge.
What specific areas of your project would you like help with?
I would like to know if the use of rem in CSS is correct, as I studied according to the guidelines provided by the platform's developers.
Yanna• 120@yannancls
Posted
Overall, the expected design appearance was properly executed, and the code was readable. Tags were used appropriately, and the website is responsive. Good work!
Marked as helpful
0- logic23dev• 70
@logic23dev
Submitted
Yanna• 120@yannancls
Posted
The code was readable and well structured. It really did what was said in the instructions, good job!
0
