I would like our feedback :)

I have just finished this challenge and I would like to know how I can improve it. One of the things I want to change later is to make the Nav stay static and not move with the page.

Any other suggestions?

Thank you!

1 Like

A responsive page would be better! I’m looking at it on iPad and the images are piled up on top of each other or covering your contact links, depending. Bootstrap would be the way to handle that.

Thank you Hilary! I thought I was using Bootstrap, but I messed up somewhere. I will go back and try again.

Hi. I played around with your code. First of all I restructred your projects section a little bit. Secondly I gave your images a width and height attribute of 300px.

<div class="row"> `

`
` `
Project 1

` `
Project 2

` `
Project 3

`
I then added some bottom margin 20px to your first jumbotron image. I then added some margin top 20px to your second jumbotron image. I then commented out your CSS for img tag. Try these changes , let me know what you think.

P.S. Since your are hosting your images on cloudinary, you can resize them before linking to them. Then you wouldn’t have to add the width and height attributes.

This is a good practice, as it also reduces initial load times for users.