My portfolio page. Need some feedback

My portfolio page. Need some feedback
0

#1

Hello fellow campers,

This is my personal portfolio page.

I must confess it took me too long getting started, but I hope you can help me review it.

Thank you :slight_smile:


#2

Looks clean and professional, good work :slight_smile:


#3

Thank you :slight_smile:

I’m happy and honoured to hear that.


#4

I’d change font-color to something whiteish because, as of now, it’s barely readable.


#5

Some of the text in the ‘About Me’ section overlaps the bottom of your photo as the view shrinks. Add about 5px to the bottom margin on the photo and that’ll take care of it.


#6

Hi,

It’s much better than mine! But there is one thing that might be worked on.

When you resize the browser, the text that starts with “Hi, I’m…” gets smaller, and then jumps up a size and starts getting smaller again. There is some code you can find that resizes your text as the screen size changes and its something like “lg-4 md-6 sm-8 xs-12”-this tutorial explains it: https://www.youtube.com/watch?v=gqOEoUR5RHg

Also, when you click the links through for contacts and portfolio, the white font disappears on the background. There’s a navbar style that will eliminate the problem…I think ‘navbar inverse’

thanks and good luck!


#7

Thanks qubkon,

I agree with you, white makes a better contrast to the background color.

I changed the font-color to white.

I appreciate your feedback.


#8

I Hello @rchristian, I noticed it earlier and gave the image a bottom margin of 15px, but its obvious that wasn’t enough.

I added about 10px to the bottom margin instead.

Thanks for the feedback.


#9

Hi, @turbokungfu,

While I was able to get the issues raised fixed, I still think I can do it better, as I have a lot of code repitions in the media queries and the selectors in the css file

Please kindly go through my css media queries and advise how I might make it better.

I really appreciate the feedback.

Thank you.


#10

Ha! You’ve busted me. I must’ve sounded more advanced than I am. I don’t even know what CSS media queries are. Your profile page is much better looking than mine. I’ll google a bit and see what I can do!


#11

Ok, under the iphone 6 thing-I just changed it to this: @media screen and (max-width: 767px){

and it worked on my browser, but I don’t know how it’ll work on the phone.

Cheers


#12

@turbokungfu thanks a whole lot.