@bdal90
Frieden
@Amanmawar17All comments
- @Amanmawar17
Hey, You replicate the design really well but it's a bit small, I'll suggest that you try to use the max-width property for your outer divs. and add some padding. Second, while you using media with a max size of screen 375px in line 191 of your CSS file in the GitHub repository change it to min size as it becomes very operational and easy. Classes in your HTML file look OK what you can do to improve it is that you can avoid multiple span tags from the summary section.
Marked as helpful - @aweezzy@Amanmawar17
Hola amigos! try to add max width to class container. and half width to image div and text div. Your submission looks good on mobile device. Thank You .
Marked as helpful - @Mahmoudamin11@Amanmawar17
Heyy! buddy, there is more margin between the paragraph and the heading, what you can do that reduce some margin and increase the font size of the heading? In content, every element should take the same gap as of other elements. You can achieve this by giving content class div tag, grid with gap property of 15px i guess and include button in it. About your button remove div tag and img tag from it and add span tag and put your svg in it and then write ""Add to Cart"" and apply margin right to span. Thank You .
Marked as helpful - @Aazeez257@Amanmawar17
Hola amigos! two problems first in summary description remove border and second button have some padding. A quick tip that if you adding bottom margin to every description item than what you can do that apply margin in y direction to the middle element it sets perfectly. Thank you!
Marked as helpful - @JoelHernEsp@Amanmawar17
Hello, buddy! your add to cart it should be a button, not an anchor tag, and you should reduce the font size of the heading when it's on responsive screens. Also what you can do with images that don't put them as background img but in img tag under div tag make them hidden or visible with media query. Good Attempt by the way. Thank You.
- @Ciensprog@Amanmawar17
It's pixel-perfect, nice job man! but the heading color should not be black. Thank you.
- @fmanjencic@Amanmawar17
Hello, Hope you doing well. In your submission, there is a minor problem your button is a dice icon and it should be the whole button with padding. The classes you give to class imgbox it should be given to the button not to div then it works well. Right now, when I click on the icon then it fetches the advice. Second, Your body should have height of 100vh then apply classes of flex, justify center, items center so your container comes in center.
Thank You.
