Abreham Nigussieβ’ 470
@DreamCoder7
Posted
Hi, system-freshπ
Good effort on this challenge!π
Here are a few things I'd like to suggest,
- Perhaps using a little bit of semantic HTML help you with accessibility and much easier to read. like wrapping the hole code into
<section class="productbig">....</section>
because it's more sense to have it's own section. And also wrapping the testimonial part like this is also practical
<figure class="box">
<div class="profile"></div>
<blockquote>
<p>...</p>
</blockquote>
</figure>
- Instead of using the
<img src=".." alt=".."/>
to display the icon-star.svg you can use the cssbackground-image
to avoid the repetition you made on HTML.
background-image: url('..');
background-size: 20px 20px;
background-repeat: repeat-x;
background-repeat: space;
....
- You already set the
padding: 0;
and 'margin: 0;' using the universal selector (*
) to reset all the element means any style that you put in the body gets inherited you don't need to repeat into thebody{}
tag
Hope you find this helpful.π
Keep coding!π
0
system-freshβ’ 140
@System-Fresh
Posted
@redstorm-hub thank youπ
0
Abreham Nigussieβ’ 470
@DreamCoder7
Posted
@System-Fresh
No problem!π€
0