@Kamasah-Dickson
Posted
Hi there👍 you did great on your solution but i suggest.
-
You use a max-width on the parent container of about 700px...you decide. The widths are spanning the whole screen that is why you need a max-width.
-
Use margin-inline:auto or margin:0 auto to center it in case you use the max-width.
-
To help with the accessibility issue, the should be a level one heading which is an h1 in your solution.
-
Also there should be a semantic tag in your solution such a main, aside and section. In your case I suggest you use the main to wrap your solution or change the div of the parent element into main
Good job and happy coding👍
@TheN0rt
Posted
@Kamasah-Dickson, thanks for the advice, but I decided to make the most of the space and not put a limiter at 700 px. Basically in such projects I practice js more than styles, so there are a lot of mistakes in styles.
