Skip to content
OverviewSolutions
17
Comments
5

Gideon Olumide Dada

@Olumide2596240 points

I am self-taught (learning), I like responsive website design and try to write cleaner code.

I’m currently learning...

Javascript, ReactJS.

Latest solutions

View more solutions

Latest comments

  • Gideon Olumide Dada•240
    @Olumide2596Posted over 2 years ago

    Hi, there, I went through your code and it seems you've not got to learning flexbox or grid yet. if you have, it will be nice to use it for this project, if not, nice job pulling this off, and be sure to try it again when you do learn flexbox.

    Also, it's better to wrap all the divs in a container like this:

    <body>
             <main>
                  <div class='container'>
                  </div>
            </main>
</body>
    0
  • Gideon Olumide Dada•240
    @Olumide2596Posted over 2 years ago

    Hello, completed this challenge yesterday and got some reviews that help me improve my code. This is the only part of the review I got that o think might be helpful to you.

    You should use the <main> element only to wrap the main content of the page. You should keep the attribution content in a <footer> element and out of the <main> element. These are semantic elements and are used to structure the page. You can read more about them here

    0
  • Gideon Olumide Dada•240
    @Olumide2596Posted over 2 years ago

    Thanks a lot, I will work on it using your feedback

    0
  • Gideon Olumide Dada•240
    @Olumide2596Posted almost 3 years ago
    1. Try a margin-left of -^px(minus). then an overflow: hidden on the parent container.
    0
  • Gideon Olumide Dada•240
    @Olumide2596Posted almost 3 years ago

    i did a margin-left of -(minus) and overflow: hidddeon; on the parent

    0