Xxxx #12
Xxxx #12
Conversation
|
index.html line 31. This is not working, the header image is not at the top and it is not “fixed”. |
This space was bumping the header image down.
I removed body padding.
|
I fixed a few of the spacing issues with the images in main.css and index.html. moving the header image below nav bar might look cleaner. |
| <p class="browserupgrade">You are using an <strong>outdated</strong> browser. Please <a href="http://browsehappy.com/">upgrade your browser</a> to improve your experience.</p> | ||
| <![endif]--> | ||
| <nav class="navbar navbar-inverse navbar-fixed-top" role="navigation"> | ||
|
|
||
| <!-- suggest moving image below nav bar. --> |
There was a problem hiding this comment.
I agree. The navbar should remain at the top of the page. I have no preference, fixed or floating.
| <img src="the-poetry-cafeHEADER.jpg" alt="Header" width="100%"> | ||
| </div> | ||
|
|
||
| <nav class="navbar navbar-inverse navbar-fixed-1000px" role="navigation"> |
There was a problem hiding this comment.
I would recomment adding a .navbar-fixed-top to the container to fix it to the top of the page. I think this looks best visually. The body padding would not need to be changed to accommodate.
|
I would advocate to merge this pull request and get the structure of the page in place. Then it would be easier to workaround some of the header issues. |
I put in a header image and background image.
I moved the nav bar to below the header image.
I changed the opacity of jumbotron and added backgrounds to the three headings.