Design comparison
SolutionDesign
Solution retrospective
Both Suggestions & Criticisms are Highly Appreciated ❤️
Community feedback
- @Maxwell-ihiasoPosted about 4 years ago
Hi Joy,
Your solution is almost good, how about making a few adjustments?
Mobile Version
- The image seems to take up so much space once the viewport is scaled up. Try defining a fixed
width
andheight
. - the
text
class could need some more padding.text-1
,text-2
, andtext-3
appear to be to close to thecontainer
. - The active state for the share button is expected to fill the bottom of the container on click. However, you have it inherit the desktop
active-state
view style. Try using positioning (parent - {position: relative}; child - {position: absolute}) to have it put in place. Use the necessary parameters to have it meet your taste. - Also reduce the opacity for the
SHARE
in the active state. - The active state container could use padding to make it nicer.
1@JoyShahebPosted about 4 years agoThank you soo much Maxwell, I really loved your feedback, and I've learned some things from you too, I really appreciate your Feedback. Take ❤️
0 - The image seems to take up so much space once the viewport is scaled up. Try defining a fixed
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