Skip to content
OverviewSolutions
8
Comments
25
Ilan
@iKnowMagic

All comments

  • Miguel Prada•170
    @mapra99Submitted about 4 years ago

    Hey! I made this mainly with the purpose of practicing React, more precisely using Hooks and function components. I'd appreciate any feedback on the logic I have there :)

    React JS site using Hooks

    1
    Ilan•675
    @iKnowMagicPosted about 4 years ago

    Good job!

    It looks like the part of the header that displays the IP Address, Location, Timezone and ISP is missing. 🤓

    I may also suggest to look into the accessibility and HTML issues from the report above.

    Take care! 😄

    1
  • Ilan•675
    @iKnowMagicPosted about 4 years ago

    Hi Connor,

    Thank you for the great feedback.

    I corrected the issues (hover effect on the header navigation, using placeholders in the form fields).

    Great suggestion to add transitions! I added transitions to the links, buttons and form fields.

    Many thanks! I very much appreciate the time that you took to review the project.

    1
  • Ilan•675
    @iKnowMagicPosted about 4 years ago

    Good job!

    You may want to consider looking into the accessibility and HTML from the report above. :)

    0
  • Ilan•675
    @iKnowMagicPosted about 4 years ago

    Good job!

    There seems to be some weird left-right scrolling in both the desktop and mobile versions.

    I'd also suggest to look into the accessibility and HTML issues as per the report above.

    Thanks for sharing. Keep up the good work : )

    0
  • Ali Gamal•295
    @ArigatouzSubmitted about 4 years ago

    it would help me so much if you give me some feedback and if there is changes need to be done thanks to you all what an awesome community

    HTML + HTML5 +CSS +SCSS +JS

    1
    Ilan•675
    @iKnowMagicPosted about 4 years ago

    Very nice job!

    I'd suggest to look into the accessibility and HTML issues as per the report above. :)

    Thanks for sharing and keep up the good work!

    0
  • Ilan•675
    @iKnowMagicPosted about 4 years ago

    Good job. I'd suggest to look into the accessibility issues above.

    0
  • Eyob tadele•85
    @hjob-1Submitted about 4 years ago

    heyy dev's people am a new web developer and its my first finished project ever .it really means alot if you guys give me comment and feedback.! #thanks

    only using html and css properties.

    1
    Ilan•675
    @iKnowMagicPosted about 4 years ago

    Good job! Thanks for sharing.

    I'd suggest to look into the accessibility and HTML issues shown in the report above.

    0
  • Ilan•675
    @iKnowMagicPosted about 4 years ago

    Great job! Thanks for sharing. I'd suggest to look into the HTML report. It seems that the "Make Reservation" form needs some work on mobile. See: https://ibb.co/fNXk2nz

    1
  • Ilan•675
    @iKnowMagicPosted about 4 years ago

    Good job! It seems that the mobile version is missing. I'd suggest to look into the HTML issues also.

    0
  • Anna Lapina•125
    @ALapinaSubmitted about 4 years ago

    Hi! I'm a complete beginner in frontend and this is my first completed challenge. So I would be happy for feedback. What do you think? What is good in my solution. And what's needs to be improved.

    Testimonials Slider with css grid

    2
    Ilan•675
    @iKnowMagicPosted about 4 years ago

    Good job! 🎉 I'd suggest to look into the accessibility issues.

    1
  • Ilan•675
    @iKnowMagicPosted about 4 years ago

    Good job!

    1
  • Ilan•675
    @iKnowMagicPosted about 4 years ago

    The solution and the design look a bit different but you are getting close. Keep up the good work!

    1