Looks good and reads well but you are failing one of the tests (#4) and could do with a bit more work on the padding when viewing on smaller screens (check out the developer tools in the browser and see how it looks on the iPhone 6 screen size as an example).
Happy coding and look forward to your next project
Hi @preciouskk, your page looks good. There are some things you may want to revisit;
- codepen provides validators for HTML, CSS and JS. Click on the down arrow in the upper right of each section and then click on the respective ‘Analyze’ link.
- In HTML you’re using some elements in a way that are not valid
- In CSS you have a typo,
(should probably have been
- as mentioned, you’re failing one of the user stories. You can click on the red failing test for an explanation of what is being tested and what part is failing
I would increase the font size and line height.
I’d remove the width: 70% on the
.containerwhen the page is stacked, using the max-width: 800px media query.
Maybe consider making the nav links stand out a bit from the body text by increasing the font-weight/font-size on them a bit.
You need to let the code wrap otherwise you will get an overflow. You can have a look a the white-space CSS property.
The images are not responsive and will overflow, you can use the css from the make an image responsive challenge.
Thanks for your feedback, will definitely implement your suggestions.
Thanks for your feedback,l will definitely look into your suggestions and implement them.