css flex

Community feedback

• 

  Angel M•370

  @csmurilloPosted almost 2 years ago

  Hello Elvis, I think you are a beginner, I was also once a beginner so I know. Here are my recommendations:

  1. Learn and follow the Mobile First Approach, this will help you so much when building web applications. DONT START BY BUILDING LARGE TO small screens. why? because when you actually build a website you have to be organized and it is more time efficient

  2. Learn semantic HTML. Learn when to use <header>, <nav>, <main>, and more.

  3. Learn to use CssGrid, this is used when certain position of content is to complicated to style with flexbox.

  Now for my two recommendations for this project:

  1. The Header <header> I did not see this tag anywhere in your project. But typically this is to wrap the top most content of the page. Typically a logo and links, hamburger menus.

  For this project the header should wrap the W and the links. Which in your case would be:

  <header>
            <h1>W</h1>
            <h3 class="menu>...</h3>
  </header> 
  

  2. The Menu <h3 class="menu">...</h3> should be

  <nav>
            <ul>
                      <li><a>Home</a></li>
                      <li><a>New</a></li>
                      <li><a>Popular</a></li>
                      ...
            </ul>
  </nav>
  

  The <nav> tag is used for to wrap links, and the <ul> is widely used with nav tag since it is a list of links.

  0

  ---

  

  Elvis•220

  @ElvislexPosted almost 2 years ago

  @csmurillo thanks, i really appreciate this. i would love it if you guide me more on this

  0