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.
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
Hello leannelafg is nice, u should put some media query for responsive and it would look better
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
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.
At 767px screen width, the logo overlaps your nav items.
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.