DrSpock123 / prj-rev-bwfs-dasmoto

0 stars 2 forks source link

Semantic HTML #4

Open aubreywullschleger opened 5 years ago

aubreywullschleger commented 5 years ago

Great! Great job using the semantic HTML element <header>. Using semantic elements helps make your HTML more meaningful to browsers reading your markup, helps with SEO, and improves the accessibility of your page. Here's info on how semantic elements help with accessibility: https://webflow.com/blog/html5-semantic-elements-and-webflow-the-essential-guide (scroll down to the 'So what are semantic elements good for?' heading) and a tutorial that goes over semantic elements and more reasons to use semantic HTML: https://internetingishard.com/html-and-css/semantic-html/.

Example: https://github.com/DrSpock123/prj-rev-bwfs-dasmoto/blob/master/Dasmoto's/index.html#L10-L12

Here, consider using the more semantic <section> element for the brushes, frames, and paint sections. Each of these sections have their own heading elements (<h1> - <h6>) making them a great use case for the <section> element (and <section> would be more meaningful than using <div> elements). See here for more info: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/section

ghost commented 5 years ago

Can I work on this one?