Skip to content
OverviewSolutions
6
Comments
2
Biruk
@BirukAbreham

All comments

  • Biruk•200
    @BirukAbrehamPosted almost 2 years ago

    Hello Mohamed,

    You did an excellent job bravo! Here are some suggestions

    • On the country flag cards you could make the entire card clickable ( At the moment only the country name is clickable)
    • You can retain the state of the theme (dark/light) in the browser storage so that when the users refresh the page, they don't lose the theme they chose. ( I suggest you use Local Storage to keep the state of the theme)
    • To get ride of "Page should contain a level-one heading", wrapper the React root div with main tag
    1
  • Biruk•200
    @BirukAbrehamPosted almost 2 years ago

    You did a great job on the UI and the responsiveness! Here are my comments

    • The form validation in the first step is not added (try to add focus to the inputs for better UX)
    • You managed the states on every step, however, you didn't calculate the final summary payment Other than that, you did a great job
    Marked as helpful
    0