Let's discuss your "Personal Portfolio Webpage"

Thanks! I think I know why the button’s not working. I’ve had trouble with the scrollable anchors in the past and it sounds like I still to need to improve them. I really appreciate your feedback.

1 Like

Hi, I’m actually interested in feedback on the code itself, if that’s even possible at this point. I started with a template, but decided to try to re-create the style very simply from scratch.

Thanks in advance :sunny:

https://codepen.io/leannelang/full/YaXyXE/

Hello leannelafg is nice, u should put some media query for responsive and it would look better

1 Like

Thanks for the note. I actually am using viewport meta tag, but fixed the sidebar at the last minute and didn’t check mobile. I appreciate it!

Your welcome and now look much better (=

Hello Everybody!

Edit: This is the latest/newest version of my website, built in Gatsby!
Live Website - https://www.davidpnowak.com


The below is with regards to my old design…

Note that some of the functionality or features aren’t working in codepen, or I had them disabled. I also used some external sources to further expand or solidify my understanding of HTML, CSS & JS. Anyways, here is my work.

Codepen Project Link (2nd revision) - https://codepen.io/Confidenceiskey/full/KQZKZz
Codepen Project Link (1st revision) - https://codepen.io/Confidenceiskey/full/yvZKBW

Project Link - https://codepen.io/miriad/full/zWGBjw/

Any comments much appreciated!

Hi coders, Just finished my portfolio challenge in freecodecamp. Pls give me your feedback.

Here’s the code:
https://github.com/ErickMurage/Portfolio

Here’s the site:
https://erickmurage.github.io/Portfolio/

Hey there @Egarum !

I think that looks great! One thing you might note is that the logo seems a little large and exceeds the navbar’s border a bit.

FCC_Screenshot

At 767px screen width, the logo overlaps your nav items.
FCC_Screenshot_1

You might want to try resizing the image and incorporating Bootstrap Navbar classes like .navbar-nav and .navbar-right to your Nav items instead of float: right. and use the .navbar-brand class for your logo (refer to 3.3 or 4.0 depending on the Bootstrap version you use) for better mobile responsiveness.

I hope that helps! I’m pretty new to coding so you might find that you have a better solution to fix your problem as opposed to mine.

Best of wishes!
Ryan

Thanks for pointing that out. Having the logo go past the nav bar boundary was actually a design feature. However, I didn’t take into consideration how it would appear on a smaller screen. I’m also a beginner, I’ll try to implement your suggestion

Note that I’m closing this thread. It’s much easier to get feedback if you create your own individual post requesting feedback for your own specific project.