Skip to content
OverviewSolutions
9
Comments
3
Henry Seigbein
@hensco95

All comments

  • Henry Seigbein•190
    @hensco95Posted 10 months ago

    Hello Ahmed. Amazing work, its really accessible. However I noticed a few things, whenever I expand a section from the accordion, the button(an image I suppose) vanishes, leaving only a broken img and an alt text of "mir". Asides that, you did well.

    Marked as helpful
    2
  • Henry Seigbein•190
    @hensco95Posted 10 months ago

    Hello there, great job on your solution, I'm currently learning react js myself. I however noticed a few things that needs to be tweaked. The texts from the metro / gaming section overlap each other for the mobile view. For the PC view, the read more button is a bit out of place. There's also a little fix needed for tab view as well. Other than that, everything works just fine. Kudos

    0
  • Henry Seigbein•190
    @hensco95Posted over 1 year ago

    Great Job Denis. I especially liked the valid format you used that validates the email as the user types. How did you implement that? Kudos

    0