All comments

• 

  Eduardo Arakaki•130

  @edukakiSubmitted about 2 years ago

  I runway a little bit out of the scope but I'm satisfied with the result. Do you have any recommendation?

  

  HTML, CSS, React, react-router-dom

  2

  60
  

  Phillip•150

  @philliplam8Posted about 2 years ago

  Love that you extended this to an entire catalogue!

  Might I suggest:

  • making the navbar sticky on the page so it always remains at the top even when scrolling
  • adding a conditional CSS property overflow: hidden to the document body when the hamburger menu is open to prevent scrolling

  0
• 

  Mahii•90

  @mahendrameghwalSubmitted about 2 years ago

  give me feedback about my project feedback will be appreciated......!!

  

  REST-Countries-API-with-color-theme-switcher

  1

  50
  

  Phillip•150

  @philliplam8Posted about 2 years ago

  Nice work building this without any frameworks! I like the Dark <-> Light mode transition animation!

  Code Suggestions:

  • Consider separating the logic into multiple module files instead of having everything within a single JS file

  Some minor stylings suggestions:

  • add some spacing between the field name and the field values
  • on the main page: -- use a lighter font weight for the field values -- add some left padding within the search input

  0

  This comment has received 1 replyShow