@andrew-g-ayad
Posted
You already did a great job creating root-level custom properties. I recommend you to do the following:
- don't set heights unless you need to
- don't use id's to style elements (it works but is not a best practice)
- use more semantic HTML (for example you could use the
h1
tag for the title) - Double-check width values in the design
I highly recommend you give these a go before moving to more advanced tools like sass
@Amit-Mahi
Posted
@andrew-g-ayad thanks for the great feedback.
- I was actually having problem with the heights i guess you solved them.
- for the styles i will use the class for now.
- for the Semantic HTML first i was thinking to use h1 but i don't know why i didn't do that.
thanks again I really appreciate the time you took and write very helpful feedback. I hope i will face more error but not these ones.