Photosnap Site using React and SCSS with animations.
Design comparison
Solution retrospective
Please let me know what you think. All I have left to do now is clean up my code.
Community feedback
- @DiarrahPosted about 4 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
2@chri55Posted about 4 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!
0 - @tarasisPosted about 4 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 😆
0
Please log in to post a comment
Log in with GitHubJoin 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