@eddybpro
Posted
Hi, CocoShesh
Congratulation on finishing this challenge, I hope it was fun.
For code here are some suggestions that might help improve the code:
- HTML code:
<div class="star-container"> </div>
- CSS code:
main{
*/ remove the scrollbar in big screen*/
overflow: hidden;
}
.best-tech{
padding-left:.25rem;
}
.star-container {
width: 6.25rem;
height: 1rem;
display: flex;
justify-content: center;
background: url(./images/icon-star.svg);
background-position: center;
background-repeat: space;
}
Tip
The value of the property font-whight
does not have a unit.
If the value of margin, padding
is 0 you don't have to add a unit.
-Examples:
h1{
font-whight:700;
p{
padding-top:0;
I hope my suggestions were helpful.
Happy coding
Marked as helpful
@CocoShesh
Posted
@eddybpro Thank you so much for your kind words and congratulations! I really appreciate your encouragement.
The challenge was indeed a lot of fun, and I'm thrilled to have completed it successfully.
I'm grateful for your suggestions to improve the code. Modularity sounds like a great idea, and I'll definitely work on breaking down the code into smaller, reusable functions or modules. This will make it easier to manage and maintain in the long run.