@SzymonRojek
Posted
Hi Lawrence,
Well done! I like your RWD and generally final solution! cool!
I have checked your HTML structure and a few points for you:
- I'd like to recommend reading more about semantic tags and some of them you can try to implement here. Divs are semantically inert elements — elements that don’t really do or say anything, of course, they are important too. You can change a bit the HTML structure by using semantic tags;
- This is only a small component but you can also use the tag h1. Just to let you know, you should only use one h1 per page. Using more than one will not result in an error, but using only one is seen as a best practice. It makes logical sense => h1 is the most important heading, and tells you what the purpose of the overall page is (generally please read about headings h1-h6);
- BEM: I am thinking about your BEM naming here (I will get back to you), in the meantime, please have a read these articles
- CSS-tricks: BEM 101;
- Smashing Magazine: BEM For Beginners: Why You Need BEM;
- There is good practice do not write styles on tags (check your CSS file => headings).
That's it from me. Please read these articles and let me know what do you think about the BEM.
Ps. Don't forget to upvote any comments on here that you find helpful.
Greetings :D
@LawrencePryer
Posted
@SzymonRojek
Thank you again for the very helpful comments. I will have a look into BEM a bit more and it was very useful to be reminded about semantic tags.
Lawrence
@SzymonRojek
Posted
@LawrencePryer
Thank you so much. I am glad. I have commented your last project again, check it out, hope you like it.