@PhantomOz
Posted
Hello Youtubbeh, fantastic job by the way.
I noticed you tried writing semantically good job. Few things I will suggest:
- You change the section element class "plan" to a div - this will make your code more semantic and fix the HTML issue you have, and also you can't have a section element without a heading element.
- To fix your Accessibility issue, you should put the div with class attribution in a footer element or in the main element.
- Also I think you should try to make it responsive for mobile devices using @media-queries in CSS.
I hope this feedback was helpful 👍
Marked as helpful