Finished Beginner Front-End Development Projects. Need Feedback!

Ok so I finished both of my front-end development projects in about 5-10 hours and I will want to know if I did good for a beginner.

Here is my tribute page: http://codepen.io/DavronUsmonov/full/mRWOpV/

Here is my portfolio: http://codepen.io/DavronUsmonov/full/VPbPWj/

Also can you help me with the portfolio. On the navigation bar there is a bit of white on the bottom and I can’t get rid of it. Thanks!

Edit: Btw I have not made them responsive so it is recommended you view them on a desktop. I will make them responsive tomorrow when I understand bootstrap more. Also you guys have any tips to make my websites better.

1 Like

One of the answers here will probably work for your white line under the navbar issue.
http://stackoverflow.com/questions/34573447/how-do-i-remove-this-white-line-from-my-bootstrap-navbar-and-change-the-text-col

I think so we can try

Here are some suggestions. Hope these will help you. :slight_smile:
The first thing, You have missed an opening tag for the conclusion Div. And have put the class in the closing Div of Timeline.
Change This

 </div class="text-center conclusion">

To This

</div>
 <div class="conclusion">

You have added too much unnecessary margin to left. I have set those margins.
Add the below three CSS rules in your stylesheet

    * {
        margin: 0;
        padding: 0;
    }            
    body {
        margin-top:60px;
        margin: 0 auto;
        width: 100%;
    }
    .container-fluid {
        margin: 0 auto;
        width: 80%;
    }

Change The intro To This

   .intro {
      border:5px solid black;
      box-sizing:border-box;
      display:inline-block;
      padding:50px;
      margin-left:40px;
      padding-bottom:10px
      }

Change the Unorderd list From

.timeline ul{
    margin-left:250px;
    margin-right:250px;
}

To

.timeline ul li {
    line-height: 26px;
}

Add another decleration in The conclusion to align it to center.
Like This

.conclusion {
    margin-top: 90px;
    text-align: center;
}

And change the last 2 CSS Rules as below.

.quote {
  margin:30px 0;
  
}

#me {
    margin-top:50px;
    margin-bottom: 20px;
    text-align: center;
}