@itsmesrishti
Posted
Hey, straight off the bat it's amazing how close you got to the design! Flexbox is the best for centering isn't it?!
After reviewing your code there is something interesting I learned from it like using <article> for blog preview card. I'd have never thought of using it for this purpose so thank you for making me think! And you are right of course <article> can be used this way.
You might want to consider the order of heading tags. <h1> should be the first heading tag and I guess for the learning label you could use something like span.
And as for your doubt regarding class names, for such small projects you can avoid giving a class name to everything. You can avoid giving a class name to <h1> because a page is only supposed to have one <h1> so just target h1 directly.
You could target h2 tags in the project without using class names as both the tags are present in different sections so just use class name of section followed by h2 (eg: .article_information h2). Again since this is a small project this h2 thing can work but maybe for bigger projects it might be better to assign a class name (like you did) to the learning tag so that if you create same style of tag somewhere else in the project for some other purpose.. you'd be able to reuse the style assigned to it. I hope that makes sense!
Again amazing work! You got so close to the design! And also thank you for teaching me something!
@CecDorWEB
Posted
@itsmesrishti Oh thanks you very much to took time to watch my code !
Your response is very interessant, I understand what you mean about titles. So I need to start with a h1. And here I don't need to use an h2, just a span it's ok.
And thanks for the explanation abour class name. I understand that if it's a small project I should keep it simple. Simply use class names if there is a risk of conflicting whit another part of the html code.
