All comments

• 

  JiawenZhuo•50

  @JiawenZhuoSubmitted over 2 years ago

  Hello, this is my first react app with Styled-component. I am a little bit unsure about how to name each folder and the hierarchy of the folders. Any critical comment will be appreciate !

  

  Responsive landing page of ecommerce- shoe store

  1

  50
  

  Akhilraj Nambiar•210

  @AkhilrajNambiarPosted over 2 years ago

  There are some issues in your report @JiawenZhuo . Here is what you could do to solve them:

  • For accessibility issues:

  1. Add the alternate text for all images you use.
  2. Give the root element of your document a class name of 'main', or else a role of 'main'.

  • For HTML issues:

  1. font-style should be replaced by font-weight, if you are trying to make the text bold.
  2. margin and padding cannot have value of none. Use 0 instead.
  3. flex-direction can only have values row, row-reverse, column, column-reverse. If you want to enable wrapping, use the flex-wrap attribute.
  4. border-color can only use colors(none is not a color!). If you don't want a border color, use transparent.

  1

  This comment has received 1 replyShow
• 

  Mohit kalme•150

  @mohitkalmeSubmitted almost 3 years ago

  Feedback on any kind of improvements is appreciated

  

  Responsive REST Countries API with color theme switcher using React js

  2

  61
  

  Akhilraj Nambiar•210

  @AkhilrajNambiarPosted almost 3 years ago

  Looks amazing and works the same way! If you were to sort the results from the search or the filters, it would be a really smooth user experience.

  Marked as helpful

  1