# Portfolio Project done! Let me know what you think

**URL:** <https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357>\
**Category:** I Built This\
**Created:** [July 1, 2018, 8:43pm UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357 "2018-07-01T20:43:18Z")\
**Posts on this page:** 11\
**Page:** 1

<div class="post-metadata">

**Author:** ![kaijamh](https://avatars.discourse-cdn.com/v4/letter/k/ea5d25/32.png) [@kaijamh](https://forum.freecodecamp.org/u/kaijamh)\
**Post date:** [July 1, 2018, 8:43pm UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/1 "2018-07-01T20:43:18Z")

</div>

[https://codepen.io/kaija-montaine/full/WyKvxx/](https://codepen.io/kaija-montaine/full/WyKvxx/)

---

<div class="post-metadata">

**Author:** ![nomanghafoor](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/nomanghafoor/32/92082_2.png) [@nomanghafoor](https://forum.freecodecamp.org/u/nomanghafoor)\
**Post date:** [July 18, 2018, 6:39am UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/2 "2018-07-18T06:39:03Z")

</div>

Your portfolio needs some improvement like making the images clickable and using proper images. However, your site is showing responsiveness . You have done a good job.

---

<div class="post-metadata">

**Author:** ![kaijamh](https://avatars.discourse-cdn.com/v4/letter/k/ea5d25/32.png) [@kaijamh](https://forum.freecodecamp.org/u/kaijamh)\
**Post date:** [July 19, 2018, 3:29am UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/3 "2018-07-19T03:29:59Z")

</div>

What do you mean using proper images? How can I do that?

---

<div class="post-metadata">

**Author:** ![nomanghafoor](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/nomanghafoor/32/92082_2.png) [@nomanghafoor](https://forum.freecodecamp.org/u/nomanghafoor)\
**Post date:** [July 19, 2018, 5:14am UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/4 "2018-07-19T05:14:34Z")

</div>

![images](https://global.discourse-cdn.com/freecodecamp/original/3X/3/1/316d4cd62fed726098a14b18e271194d211fb2f4.PNG)

For your projects like tribute page or project 2, 3 and 4 etc you ought to take the screenshots of your projects or web pages. It would be more appealing and exhibit your professional skills.

---

<div class="post-metadata">

**Author:** ![Gilbert1391](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/gilbert1391/32/93761_2.png) [@Gilbert1391](https://forum.freecodecamp.org/u/Gilbert1391)\
**Post date:** [July 19, 2018, 5:36am UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/5 "2018-07-19T05:36:03Z")

</div>

I think I saw your portfolio before and I can tell you have improved a lot. Just a few things, I think you should change the image for the tribute page because it looks like an icon out of place, instead have a screenshot for the webpage being displayed similar to the google ones.

Also I think the hero will look much better if you change `object-position: bottom;`

Also you probably want to take a look at your div with id “social” , it’s getting in the way of the email input field and the submit button due to `transform: scale();` I took a look at the div and I saw a lot of empty anchors for some reason.

---

<div class="post-metadata">

**Author:** ![PeterWebDev](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/peterwebdev/32/90389_2.png) [@PeterWebDev](https://forum.freecodecamp.org/u/PeterWebDev)\
**Post date:** [July 19, 2018, 9:57am UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/6 "2018-07-19T09:57:54Z")

</div>

Your site is has some responsive elements but it still has some side scroll going on. On my mobile the project is all in one row which is good but imo i would center them in addition. Your submit text is touching the right border, it should have equal left and right padding(tested in chrome)

Playing with the window size when the 4th project drops to the next row its seems misaligned its not aligned to the left nor center.

For links like the social icons i would add Target="\_blank" so it opens in a new tab instead of navigating the user away from the site.

---

<div class="post-metadata">

**Author:** ![kaijamh](https://avatars.discourse-cdn.com/v4/letter/k/ea5d25/32.png) [@kaijamh](https://forum.freecodecamp.org/u/kaijamh)\
**Post date:** [July 20, 2018, 4:14am UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/7 "2018-07-20T04:14:38Z")

</div>

Thank you all for your advice!

I made some edits. Could you please check out my edits and let me know what you think?

[https://codepen.io/kaija-montaine/pen/VBwwNY](https://codepen.io/kaija-montaine/pen/VBwwNY)

Thank you!

---

<div class="post-metadata">

**Author:** ![sculprog](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/sculprog/32/158756_2.png) [@sculprog](https://forum.freecodecamp.org/u/sculprog)\
**Post date:** [July 20, 2018, 12:03pm UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/8 "2018-07-20T12:03:08Z")

</div>

Hi!

One minor thing I noticed is the Submit button and its text are a little misaligned. It looks like the button is too short for the text or the text is too big for the button.

Hope that helps!

---

<div class="post-metadata">

**Author:** ![kaijamh](https://avatars.discourse-cdn.com/v4/letter/k/ea5d25/32.png) [@kaijamh](https://forum.freecodecamp.org/u/kaijamh)\
**Post date:** [July 21, 2018, 4:44am UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/9 "2018-07-21T04:44:00Z")

</div>

Thank you! How can I make each section fit a whole page?

---

<div class="post-metadata">

**Author:** ![Gilbert1391](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/gilbert1391/32/93761_2.png) [@Gilbert1391](https://forum.freecodecamp.org/u/Gilbert1391)\
**Post date:** [July 21, 2018, 4:50am UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/10 "2018-07-21T04:50:49Z")

</div>

You would use `height: 100vh;` vh stands for viewport height.

---

<div class="post-metadata">

**Author:** ![ILM](https://sea3.discourse-cdn.com/freecodecamp/user_avatar/forum.freecodecamp.org/ilm/32/363239_2.png) [@ILM](https://forum.freecodecamp.org/u/ILM)\
**Post date:** [June 1, 2021, 1:47pm UTC](https://forum.freecodecamp.org/t/portfolio-project-done-let-me-know-what-you-think/205357/11 "2021-06-01T13:47:01Z")

</div>


