@Tielinen
Posted
Hi Taha, Good work.
You captured design well. Your code is short, effective and easy to read.
Few minor details, I would,
-
focus more about responsiveness to make sure that the page looks the way I want also between breaking points, for example in tablets.
-
change class naming convention, more general, "font-700" to "main-weight". If the design is adjusted later, you don't need to change the HTML, just a value in CSS.
-
keep styling altogether away from HTML, by naming classes about what they are. For example"main-heading. "card-heading" and then style them in CSS, example:
.content,
.rate,
.name {
font-weight: 700;
}
Like mentioned before, these are just minor details. For my eye goof quality code. Keep on coding, you are getting good at this.
Best wishes Janne
@thaykrgl
Posted
@Tielinen Thank you very much for your suggestions. They are very important to me.