Please provide feedback on my landing page project

Hi everybody
I am finished or am in the process of finishing my Landing Page project and would like to know your thoughts.

Just have the table to sort out and a few minor issues.
many thanks

https://codepen.io/lukemersh/full/EJREaJ

no sure whats going on with that extra column in the table though.

The last tr element has 4 tds meanwhile other trs have 3 tds.

brilliant many thanks, sometimes it takes an extra pair of eyes to spot the error :+1:

bloody video links are driving me crazy!!!
i cant get the mp4 to work with a mp4 on my desktop :frowning_face:

I think you have to use the embed code they give you.

<div style="padding:56.25% 0 0 0;position:relative;">
  <iframe id="video" src="https://player.vimeo.com/video/333838336?title=0&byline=0&portrait=0" style="position:absolute;top:0;left:0;width:100%;height:100%;" frameborder="0" allow="autoplay; fullscreen" allowfullscreen></iframe>
</div>
<!--  Put this before the closing body tag or use the Codepen Settings button and add it on the JavaScript tab -->
<script src="https://player.vimeo.com/api/player.js"></script>

The above embed code is set to be responsive, if you want to limit the width you can give it a container and set a max-width.

<div class="video-container">
  <div style="padding:56.25% 0 0 0;position:relative;">
    <iframe id="video" src="https://player.vimeo.com/video/333838336?title=0&byline=0&portrait=0" style="position:absolute;top:0;left:0;width:100%;height:100%;" frameborder="0" allow="autoplay; fullscreen" allowfullscreen></iframe>
  </div>
</div>
.video-container {
  /* Set the max-width to whatever you like */
  max-width: 980px;
  /* Center on page and give a bit of vertical space */
  margin: 40px auto;
}

Edit: added the required id attribute

thanks but i have to use the html5 video element.
which poses a problem for me.

or maybe not :slight_smile:
i just need to give it the id tag

Thank you :+1::+1::+1::+1:

You are right I forgot about the id. Good to hear you sorted it out.

Finally got the project completed!!!

Congrats!

I just noticed the slider to turn the shoe, I like that idea.

Are you going to keep working on it? It still needs a little love with the layout, but mainly just some centering and spacing. Maybe also add the full Bootstrap nav so it gives you a mobile nav?

Anyway, good job.

No - I have just submitted it now.
I have used some bootstrap, but wanted to keep things simple and just use the stuff i have worked on.

When I freelance for clients i always use bootstrap, but not really for this project, maybe i will for the last project.