FCC Project -2 = Redesigned Personal Portfolio Page (Feedback Required )

Hello, Campers I have redesigned my personal portfolio webpage, please review and give your valuable suggestions…

Personal Portfolio Page

Instead of this,

<link href="https://fonts.googleapis.com/css?family=Dancing+Script" rel="stylesheet">
    <!-- Google Font for Social Media Section -->
    <link href="https://fonts.googleapis.com/css?family=Goudy+Bookletter+1911" rel="stylesheet">

use this

<link href="https://fonts.googleapis.com/css?family=Dancing+Script|Goudy+Bookletter+1911" rel="stylesheet">

Just to save up lines

1 Like

thanks a lot for your valuable comment :slightly_smiling_face::slightly_smiling_face:

Just a comment:
Doing something along the lines of

nav>a>img {
   css: css;
}

doesn’t look very nice. I’d be much more inclined to do something along the lines of:

.nav-link-img {
   css: css
}

And have the corresponding classes on those images.

1 Like

thanks a lot, I will take care of that :slightly_smiling_face::slightly_smiling_face:

1 Like