"Kinda" Responsive GitHub-user-search-app using HTML, CSS, JS, &React

Community feedback

• 

  Jose Truyol•50

  @xolottPosted about 3 years ago

  Hi @LonelyBuddy

  Awesome solution! I have some comments:

  • To improve the accessibility of your solution add the attribute aria-hidden="true" to decorative images like the user's profile picture and the svg icons
  • Try to use Semantic HTML. Your code should include at least a main element

  After you make those changes don't forget to push your changes to GitHub and generate the accessibility report one more time.

  I really like how it looks! Pixel-perfect solution!

  Edit:

  Just one more note:

  • Practice the separation of concerns on your code. The presentation layer (the react components) should don't know about the business logic (how you fetch the API). You can try something like Callbacks hooks in react (to begin with something) or Redux (recommended approach)

  Marked as helpful

  1

  ---

  

  Henry Zheng•420

  @LonelyBuddyPosted about 3 years ago

  @xolott Thank you so much for the feedback! I'll take a look at those notes online.😃

  0