Skip to content
OverviewSolutions
21
Comments
5

clinton wambugu

@Clinton-devKenya/Nairobi455 points

Greetings fellow humans, I come bearing gifts from Mars :-)

I’m currently learning...

React, sass and ruby on rails

Latest solutions

View more solutions

Latest comments

  • clinton wambugu•455
    @Clinton-dev

    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

  • clinton wambugu•455
    @Clinton-dev

    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
  • clinton wambugu•455
    @Clinton-dev

    Hi @iamenochlee great job on creating the app to be as similar to the design. Suggestions:

    1. You could add some margin on both the top and bottom of the page.
    2. Add some margin on the sides both left and right on mobile screens.
    3. Add more details (link to active sites, technologies used etc)in description section of your readme
    Marked as helpful
  • clinton wambugu•455
    @Clinton-dev

    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.

  • clinton wambugu•455
    @Clinton-dev

    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