# Could anyone review my first 'proper' project using HTML/CSS

**URL:** <https://forum.freecodecamp.org/t/could-anyone-review-my-first-proper-project-using-html-css/439568>\
**Category:** I Built This\
**Created:** [January 8, 2021, 6:47pm UTC](https://forum.freecodecamp.org/t/could-anyone-review-my-first-proper-project-using-html-css/439568 "2021-01-08T18:47:43Z")\
**Posts on this page:** 6\
**Page:** 1

<div class="post-metadata">

**Author:** ![jenulds-dev](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/jenulds-dev/32/198225_2.png) [@jenulds-dev](https://forum.freecodecamp.org/u/jenulds-dev)\
**Post date:** [January 8, 2021, 6:47pm UTC](https://forum.freecodecamp.org/t/could-anyone-review-my-first-proper-project-using-html-css/439568/1 "2021-01-08T18:47:43Z")

</div>

[https://codepen.io/jenulds-dev/pen/OJRwgBe](https://codepen.io/jenulds-dev/pen/OJRwgBe)

This is the portfolio project on FCC. I appreciate critical feedback so say what you have to say. I would also mention that I am about a week into this so my design skills are not yet good.

Thanks! 😄

---

<div class="post-metadata">

**Author:** ![kevinSmith](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/kevinsmith/32/72934_2.png) [@kevinSmith](https://forum.freecodecamp.org/u/kevinSmith)\
**Post date:** [January 8, 2021, 7:06pm UTC](https://forum.freecodecamp.org/t/could-anyone-review-my-first-proper-project-using-html-css/439568/2 "2021-01-08T19:06:04Z")

</div>

Cool, looks pretty good.

> my design skills are not yet good.

Join the club.

I see a few duplicate ids or “project-item” - ids must be globally unique. If you want the same thing applied to several elements, that is what a class is for. Also, not everything needs an id.

```auto
    <span id="contactl">
    <section id="contact">

```

A `section` should not be a child of a `span`. Perhaps a `div` would make more sense there.

Your footer is invisible.

You have a closing `html` tag but no opening?

I would say that you don’t need that many fonts. In general, 1 or 2 will do.

You should add

```auto
<script src="https://cdn.freecodecamp.org/testable-projects-fcc/v1/bundle.js"></script>

```

so you can run the test suite.

Still, it looks pretty good.

---

<div class="post-metadata">

**Author:** ![jenulds-dev](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/jenulds-dev/32/198225_2.png) [@jenulds-dev](https://forum.freecodecamp.org/u/jenulds-dev)\
**Post date:** [January 9, 2021, 3:00am UTC](https://forum.freecodecamp.org/t/could-anyone-review-my-first-proper-project-using-html-css/439568/3 "2021-01-09T03:00:32Z")

</div>

Thanks buddy! and yes the HTML may be a little bit messy as I worked locally and then transferred everything to codepen. As for the footer, it is invisible as I cannot figure out how to apply it in codepen. Anyways thanks for your feedback. 🙃

---

<div class="post-metadata">

**Author:** ![AndrewAung11](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/andrewaung11/32/232871_2.png) [@AndrewAung11](https://forum.freecodecamp.org/u/AndrewAung11)\
**Post date:** [January 9, 2021, 3:16am UTC](https://forum.freecodecamp.org/t/could-anyone-review-my-first-proper-project-using-html-css/439568/4 "2021-01-09T03:16:10Z")

</div>

@jenulds-dev You don’t need to create a new body tag. And if you want to import some thing there is a tool for it in codepen. Go to [this post](https://forum.freecodecamp.org/t/responsiveness-is-my-weakness-personal-project-wedding-website/436341/2) for more.

Your wed design is Great  
But  
If you combine these

 ![image](https://global.discourse-cdn.com/freecodecamp/original/3X/e/7/e770581a3e73124da235388c6ab40f97d92a8f1a.png) it will be better!  
**Hope This Help**

---

<div class="post-metadata">

**Author:** ![jenulds-dev](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/jenulds-dev/32/198225_2.png) [@jenulds-dev](https://forum.freecodecamp.org/u/jenulds-dev)\
**Post date:** [January 9, 2021, 6:07am UTC](https://forum.freecodecamp.org/t/could-anyone-review-my-first-proper-project-using-html-css/439568/5 "2021-01-09T06:07:15Z")

</div>

Hey, Thanks. I used viewport height so maybe the codepen layout affects the rendering but on typical desktop / mobile dimensions, the design looks fine I think. Anyway, I appreciate your feedback. 😀

---

<div class="post-metadata">

**Author:** ![system](https://global.discourse-cdn.com/freecodecamp/original/3X/e/1/e1c433102e34b46dfaa3e15d13ab9a514b127078.svg) [@system](https://forum.freecodecamp.org/u/system)\
**Post date:** [July 10, 2021, 6:07pm UTC](https://forum.freecodecamp.org/t/could-anyone-review-my-first-proper-project-using-html-css/439568/6 "2021-07-10T18:07:27Z")

</div>

Closed on July 10, 2021, 6:07pm UTC

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.