Skip to content
OverviewSolutions
7
Comments
4
MarcinFiuk
@MarcinFiuk

All comments

  • MarcinFiuk•270
    @MarcinFiuk

    Nicely done project. Very detailed CSS Reset

  • MarcinFiuk•270
    @MarcinFiuk

    Hi Dike Obinna Clinton

    Nice solution, everything looks very nice.

    However, I noticed you don't use button for navigation icons only div instead. If you use <button> you would be able to focus on then with keyboard and your solution will be more accessible.

    You could also try to separate navigate and buttons from hero slides and style them in relation to <section class="hero">. That will help you avoid repetition and they stay static.

    I hope you find that info useful.

    Marked as helpful
  • MarcinFiuk•270
    @MarcinFiuk

    Hi Curtis :)

    Overall, the code works how you intended it to work. However, I agree that the Figma and readme file are not clear how pomodoro works. I found pomodoro definition "The pomodoro technique is a time management method based on 25-minute stretches of focused work broken by 3-to-5 minute breaks and 15-to-30 minute breaks following the completion of four work periods". I think it should automatically move to the next step and stop when it goes the whole set.

    I noticed that when you update the color and font it doesn't apply to modal and they stay unchanged.

    I hope this helps

    Marked as helpful
  • MarcinFiuk•270
    @MarcinFiuk

    Hi André,

    Nice solution, everything looks ok, however, when I went through your preview side I noticed that when you use one filter (e.g. pick up all the countries in Europe) and next tape a name in second input (e.g. Portugal or Germany) it start infinite fetch.