Kadir Yıldırım• 130
@kadiryildiri
Submitted
P
Ivaylo Ivanov
@ValsCodes
All comments
- PIvaylo Ivanov• 70
@ValsCodes
Posted
Yes. It is accessible, some minor padding improvements can be made as well as text sizing. The layout if perfect in terms of size. The code is readable even for someone who hasn't touched vite and React. It is very close to the design with minor flows, well done!
1 - chaimae jebrane• 120
@Chaimaejebrane
Submitted
What are you most proud of, and what would you do differently next time?
I am so happy that I finished this challenge. I have learned Tailwind CSS Grid, something that was completely new to me.
PIvaylo Ivanov• 70@ValsCodes
Posted
Yes. Yes, there is a small problem with space between the price and the discount price in the mobile view and it could use a little more padding in general. Size wise everything is okay. The Code is readable, I am just still kind of new to css in general and it's hard to for me get why there are so many overwrites in the css style sheet. It is very much like the Design, well done!
Marked as helpful
1 - Aditya Borse• 270
@adiborse111
Submitted
PIvaylo Ivanov• 70@ValsCodes
Posted
The Solution does not contain sematic html. The site is accessible but the color palette is way off from the design. In terms of size it's close to the design but only for the desktop version, for mobile and other resolutions the layout is not in order. It's easy to read, can be reused and is structured. Yes.
0 - Myat Thiha Naing• 130
@MyatThihaNaing-mthn
Submitted
What are you most proud of, and what would you do differently next time?
The hardest part was the responsiveness of the website. I did my best although the current solution is not optimized for responsiveness.
What challenges did you encounter, and how did you overcome them?
THE RESPONSIVE WEB DESIGN
What specific areas of your project would you like help with?
Please give me some recommendations on how to improve my responsive design thinking.
PIvaylo Ivanov• 70@ValsCodes
Posted
1 Yes. 2 It is, the colors, fonts are a bit off. Hovering on the button doesn't do do anything, but if the user didn't have the design this is fine. 3 Size is somewhat okay. The fonts are a bit larger than the design 4 Probably hard to be reused, it's a bit hard for me read all that is going on there 5 around 30% of a difference
0 @RiccardoColtrinari
Submitted
PIvaylo Ivanov• 70@ValsCodes
Posted
Perfect styling. Sematic HTML. Minimal size differences. Exactly as the design! Solution is as close it gets to the design.
1- James Brown• 260
@jamesbrown173
Submitted
PIvaylo Ivanov• 70@ValsCodes
Posted
Does the solution include semantic HTML? -Yes. Is it accessible, and what improvements could be made? -Its smaller than the design. Does the layout look good on a range of screen sizes? -Yes. Is the code well-structured, readable, and reusable? -Yes. Does the solution differ considerably from the design? -30% diference.
0
