Skip to content
OverviewSolutions
5
Comments
2
Fabricio Ceppi Rojas
@fabricioceppi

All comments

  • Fabricio Ceppi Rojas•130
    @fabricioceppiPosted about 3 years ago

    Hey. Great job! I was just checking your code and I love the way you wrote your Js. Pretty simple, efficient, and readable. I don't have any suggestions, but I wanted to tell you that I really liked it!.

    Marked as helpful
    0
  • ArslanJ9•150
    @Arslanj9Submitted about 3 years ago

    Anything to make it better would be appreciated!

    Used Flexbox

    1
    Fabricio Ceppi Rojas•130
    @fabricioceppiPosted about 3 years ago

    Hello Arslan. Good Job! It's looking pretty good. Here are some suggestions to solve those accessibility problems: Change your <div class='bg'>...</div> to <main class='bg'>...</main>, the h3 for an h2 and the h5 for an h3, and lastly add an alt attribute to your <img src="images/icon-music.svg">. In this case the alt text can be 'decorative', but is necessary for a correct markup.

    Have a good day!

    Marked as helpful
    1