Your feedback on this project would be really helpful and motivation to progress.
Welcome to the forums @vishal3. Some things to revisit;
- verify your email with codepen so we can view your page in full view
- On using codepen. codepen only expects the code you’d put within the
<body> </body>tags in HTML. (No need to include the body tags). For anything you want to add to<head>click on the ‘Settings’ button, then HTML and add it into the ‘Stuff for <head>’ box.- as an aside it’s
<!DOCTYPE html>no need to add the number 5, this is what tells the browser the code is html5
- as an aside it’s
- Run the tests again. Your page passes 9/10 tests. Click on the red button to see what’s failing and read the failing message text.
- don’t use the
<br>element to force spacing. Use margin and/or padding in CSS - there are no such element as
<p1>,<p2>or<p3>it’s a<p>element always - why do you use an unordered list (
<ul>) and then add numbers to your list? why not just use an ordered list (<ol>)?- also, the
<p>element is not a valid child of<ul>or<ol>. you would use<li>for each of your list items. Revisit this lesson to learn about making a list
- also, the
- Review the lesson about giving meaningful text to links
- Instead of “wikipedia entry” as a link wouldn’t it be better if the link was “read more about this incredible scientist”?
- do all of your styling using external CSS. Don’t use in-line styling.
- in CSS you declare
max-widthtwice in theimg2class. That will have an effect in how your page displays depending on which you wanted to use.