All comments

• 

  P

  Guilherme Henrique Teixeira de Oliveira• 110

  @guilhermehto

  Submitted over 2 years ago

  Just any feedback would be good :) Decided to cut this one short as I was getting bored of the project lol

  cheers

  

  Devfinder w/ styled components and TS

  1

  30

  

  Salym Akhmedov• 170

  @salymk

  Posted over 2 years ago

  The project looks nice and clean. So far I found a couple of things that you should look at:

  1. When I input some value and click the Enter button it does not submit the form. I have to click on the Search button in order to submit the form. What can you do:

     • Make the Wrapper component a form instead of a div.
       • Wrapper component: https://github.com/guilhermehto/frontendmentor/blob/main/challenges/github-user-search/src/components/SearchBar.tsx

  2. When I click on a link, it does not open a new tab, my personal preference is to open a new tab.

  • For example: <a href="google.com" target="_blank" rel="noreferrer noopener"> Google

  </a>

  1. It's hard to inspect your code, I can't tell which elements are being styled because all of the classNames are not descriptive.

  • You can import styled components like this: import styled from "styled-components/macro"; in order to see descriptive classNames

  Marked as helpful

  2

  This comment has received 1 replyShow