@R3ygoski
Posted
Hello @Ephy-kimotho, good job, your project is very good.
I want to give you some tips, first about your HTML. I highly recommend to use svg's with <img/>
tag, because it's make your code more clean. And about Semantic, this part <section class="text">
could be an <article>
, and your <div class="container">
it's more apropriate be used as <main>
tag, because it's the main content of the page. Here a link about semantic HTML, W3School - Semantic HTML.
And another tip, now focused on CSS, I highly recommend to use rem units in font-size
, because it's a relative unit, it can adapt to user font size configuration, and it's help a lot in accessibility.
And again, congrats for conclude your project, it's very good. If something I said was unclear, please, comment below and I will try to help.
@Ephy-kimotho
Posted
@R3ygoski Thank you for the feedback, I will try to implement semantic HTML in my next projects.