interactive comments section

Community feedback

• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted about 2 years ago

  Hi there👍 you did great on your solution but i suggest.

  1. 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.

  2. Use margin-inline:auto or margin:0 auto to center it in case you use the max-width.

  3. To help with the accessibility issue, the should be a level one heading which is an h1 in your solution.

  4. 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👍

  1

  ---

  

  Kirill Mishin•280

  @TheN0rtPosted about 2 years ago

  @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.

  0