Tribute Page - Bruce Lee

Code Feedback
1

Just finished my Bruce Lee tribute page for my project.
The links here: https://codepen.io/nicer00ster/pen/OObEKq

Any feedback would be appreciated. Thanks!

2

While your project looks good on a wide screen (i.e. pc), on a smaller screen, but because you used an inline style of padding-left: 128px;, things look off on a smaller screen (see screenshot below). You want the text below the image to fill the screen instead of having the 128px of padding even when the screen size is narrower.

image
image533×789 64.2 KB

3

Yeah you’re right… I took off the inline padding-left: 125px; I sort of just wanted it to be centered with full screen while using PC. I’m sure there’s a way to do this just have to figure it out. Thanks for the feedback!

4

Not sure if this is the effect you are wanting, but replacing:

    <div class="col-md-8 col-md-8 col-md-10">

with

    <div class="row justify-content-center">
    <div class="col-lg-8">

and then adding a closing </div> on a new line in your code shown to the left of the comment below:

      </blockquote>
      </div>
     </div> <!-- extra closing div for new div added for class="row...." above -->
    </div>
  </div>
    <footer style="padding-left: 30px;">
  <p>Written by Alex Busch</p>
    </footer>
</div>

This will center the content and only make it take up 75% of the screen for wider screens and then full width for smaller screens.

1 Like
5

Oh wow! That’s exactly what I wanted it to look like. Thanks man! Very much appreciated. :star_struck:

6