Submitted over 3 years ago

Photosnap Site using React and SCSS with animations.

Christopher Till• 1,070

@chri55

Preview site View code

This is a solution for...

Photosnap multi-page website

HTML
CSS
JS

4advanced

View challenge

Design comparison

SolutionDesign

Solution retrospective

Please let me know what you think. All I have left to do now is clean up my code.

Community feedback

Diarrah• 3,418

@Diarrah

Posted over 3 years ago

Good job! I thought I was fast doing this project but you were super fast lol.

The 2 things I noticed is that it's a bit verbose with the stories page - I did a simple JSON & it was pretty simple with the pictures like that. Also, when mobile nav is out + you redirect to a page the nav remains out. You can make it go in with a really simple useState.

Hope this helps :)

-Diarrah

Christopher Till• 1,070

@chri55

Posted over 3 years ago

@Diarrah Thanks. For the stories page I was a little unsure what I was doing, especially since I was not able to make absolute image urls work with react for some reason. JSON was a good suggestion though, I will experiment with it and see if it does anything to reduce the code!

As for useState, I will look into it for sure. I'm new to react so I haven't heard of a lot of these features yet.

I appreciate the nice feedback!

Robert McGovern• 1,095

@tarasis

Posted over 3 years ago

Just talking visually, I like your take on the design. I think the logo is better on the left like you've done, and I like the colours in the social icons down the bottom. Although I prefer the design's bolder titles on the photo boxes.

No comment on the source, I don't know React 😆

Please log in to post a comment

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord