@csphilli
Latest solutions
Latest comments
- @Julia-Fidison
Hello, Good job. I would like to give you a few suggestion if you don't mind.
In your html code. It will be easier for you and the other if you follow the html semantics for example <main class="container"> <img> <article> <h1>.......</h1> <p>.......</p> </article> </main> Concerning the heading I think you should always start with <h1> <h2> <h3> .... You can't go to h2 after h1, they need to follow a chronological order if you want to change the font-size of your heading just override it in you css Learn more about that here And more about the html semantics
- @sadykova@Julia-Fidison
Hello, Great job. I would suggest you to import your google font like @import url('https://fonts.googleapis.com/css2?family=Outfit:wght@100&display=swap'); in your stylesheet as you already have a separate style.css.
And change the color to h1 by Grayish blue: hsl(220, 15%, 55%).
I think you should make you're site responsive by using media query as: 320px — 480px: Mobile devices 481px — 768px: iPads, Tablets 769px — 1024px: Small screens, laptops 1025px — 1200px: Desktops, large screens
You can learn more about that here
Marked as helpful - @boostbm@Julia-Fidison
Hello, that's really a great job. you have done here. I would like to suggest if you don't mind to use the html semantics like header, main, section... instead of div. So it will be easer for you and the other to read your code. Example:
<main> <header> <h1>title</h1> <p>paragraph</p> </header> <main>Marked as helpful - @Julia-Fidison@Julia-Fidison
Thanks a lot for your feedback. I really appreciate.
