It took me 3 days to finish my Portfolio . Review would be great !?!

Code Feedback
1

it took me almost 3 days to complete my portfolio and the reason is that i made this portfolio for my personal and blog use too; not only for the challenge.In the future i am thinking about adding some more interesting UI .But for now this is it.
#if you guys have any idea ,do share i’m sure gonna implement it to the page.
[Edited : fixed some size issues as pointed out in comment section]

Here is the Pen:

2

Hey @RocktimSaikia,
very nice site design. I especially liked the hovering social icon buttons at the bottom.
I would try to improve the responsiveness of the site, as it currently displays well only on large screens.

Also the image with the alt text, a standing boy, did not load for me.

2 Likes
3

sure.And thanks for the feedback.i’ll keep that in mind. :slight_smile:

4

Heya @RocktimSaikia!

It looks really good. Here’s some of the feedback that I have:

In the about me area, I would suggest making all 3 boxes the same size.
When I click portfolio, it doesn’t seem to do anything in the navigation bar.
The picture for Medium seems to be broken, at least from what I see.
Lastly, the cursive font is not the easiest to read. I would suggest to change it to another font.

I really like the social media icons on the page as well!
Hope that helps!

2 Likes
5

Good looking site! Just to let you know, when I minimized the screen, the hamburger menu didn’t work.

6

Thanks for the feedback :slight_smile: i also noticed lately that images sometimes gets broke and the reason is probably when i linked the images in code i used an online Url shorten website to make the Urls short and simple. may be that’s why the page having some problem in loading the images.

and the boxes seems to same size i guess .

and yeah i also think the cursive font was not a good idea .
I’ll fix that :slight_smile:

7

You may just add min-height:150px; to .glance-box{....} to make the boxes same height

8

oh yeah i almost forgot about that.I’ll fix it soon .Thanks :slight_smile:

9

sure but ain’t all the boxes looks to be of same height ?

box
box1298×536 88.9 KB

10

not for me:

image
image1303×379 41.7 KB

11

ohh man. okey i’ll fix that .thanks for pointing out dude

12

On smaller screen sizes things can get a little weird. This looks good though, one of the bets I have seen.

Make your navigation stuck to top of the screen, like moves with you. It’s a simple

This might be outdated, but it’s a start —> https://www.w3schools.com/howto/howto_js_navbar_sticky.asp

1 Like
13

thanks .i’ll surely fix that :slight_smile:

1 Like