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

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

#1

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

Personal Portfolio Page


#2

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


#3

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


#4

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.


#5

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