Skip to content
OverviewSolutions
2
Comments
3
Ahmed Hashem
@mohamed-H99

All comments

  • Ahmed Hashem•80
    @mohamed-H99Posted about 3 years ago

    CONGRATS! 👏 👏 Your live version's design is like 95% similar to the design given, Well done. (Page title is smaller), But everything is just done the way it should be. For the code I've seen it and the structure is good and the code is organized. Just for the programming side, You added a static data instead of using the client/user public IP! That's all i hope you are doing great and Good job.

    Marked as helpful
    0
  • Ahmed Hashem•80
    @mohamed-H99Posted about 3 years ago

    VERY GOOD!! 👏 👏 Just work on your spaces (content box - page height) and your colors (input text is similar to input::placeholder it's hard to tell the difference between them *bad UX) That's just my honest opinion. Please take this advice from me as a Friend 🤝

    0
  • Ahmed Hashem•80
    @mohamed-H99Posted about 3 years ago

    I've seen the live version, and it's pretty good! WELL DONE 👏 but there are some points on the UI such as: the search input padding - the font weight is not similar - the map doesn't have (zoom control) in the design given to copy it exactly the same For the Programming work: why adding a domain is not working?! *you can use regex (search google) for that... (Please take this advice from me as a friend)

    0