Alli Mariam Titilope• 210
@oma16
Submitted
clinton wambugu
@Clinton-dev
All comments
- clinton wambugu• 455
@Clinton-dev
Posted
Nice work love the implementation, just a minor suggestion to add on user experience- to close the cart card when user clicks outside otherwise one has to refresh the page to remove it
0 - mo-frontend• 30
@mo-frontend
Submitted
Always feedback is wellcome
clinton wambugu• 455@Clinton-dev
Posted
Just a to point a minor suggestion: you can add margin-top-right to results div. Other than that your design on both desktop and mobile look clean, Keep up!
Marked as helpful
0 - enochlee• 600
@iamenochlee
Submitted
My very first react project, completed this a week ago just didn,t submit made use of the json file.
clinton wambugu• 455@Clinton-dev
Posted
Hi @iamenochlee great job on creating the app to be as similar to the design. Suggestions:
- You could add some margin on both the top and bottom of the page.
- Add some margin on the sides both left and right on mobile screens.
- Add more details (link to active sites, technologies used etc)in description section of your readme
Marked as helpful
0 - Andrew Simons• 290
@AndrewSimons3
Submitted
clinton wambugu• 455@Clinton-dev
Posted
I found app doesn't display well when viewed from a mobile end but looks great design on a desktop . You could also customize your readme to include the tech used where you got stuck , instead of having the default one that frontend mentor provides.
0 - Gesiere• 780
@Gesiere
Submitted
Would love any critisim on this project. Thank you.
clinton wambugu• 455@Clinton-dev
Posted
The illustration box desktop isn't at the correct position, you can try transferring it to .box-image container as illustration-box is positioned relative to the nearest positioned ancestor
1
