Portfolio Webpage FCC

Hi All,

Hope all of you doing well in abundance and happy coding joy.

So i’m adding finishing touches to the portfolio webpage project as per these criteria:

https://learn.freecodecamp.org/responsive-web-design/responsive-web-design-projects/build-a-personal-portfolio-webpage/

I have added the project to Github pages which you can find via link:

https://romson.github.io/Portfolio-Webpage-FCC/

Can you please give me your honest feedback on what adjustments/improvements it needs more? Not only to pass the test, but also for future work and good practice.

Looking forward to your replies and Thanks!

Kind regards,
Romson

just curious, are any of the sentences supposed to make sense? Right now I’m struggling to read them so I’m not sure if they are just random words or you really mean something by these words. I hope I don’t sound rude. Not trying to be!

Your design skills are certainly strong and overall the website is aesthetically pleasing. Awesome work!
A few things…
Are you using placeholder text? Like the camper above noted, several of the sentences do not make semantic sense. The hamburger menu is lost (I’m assuming it’s black on black background?) when the screen is changed to a smaller size.

Beautiful work, though!

lol I thought so too when I first read it. It’s from http://socialgoodipsum.com

I like it better than the standard lorem ipsum. Check this link for some more funny lorem ipsum generators:

Have fun! :smile:

thank god for that! I thought you were somewhat crazy!

Hey, Thank you very much. That’s too kind :smile:

I have tried to change: .navbar-toggler-icon

Even with span infront (span.navbar-toggler-icon) and value !important but with no result.

The .navbar-toggler border-color does show though. Any ideas?

i also can’t seem to get the .welcome-section background image to show on mobile. I have this for media query:

@media screen and (max-width: 500px) {
.welcome-section {
background-image: url(‘images/bg-new.jpg’);
background-repeat: no-repeat;
background-attachment: fixed;
width: 100%;
}
}

Update: added the background-position: property and all is good now :slight_smile:

You can find the stylesheet on Github here:

Thanks again. for your kind words.

Rofl!

Wait till you read Bacon ipsum