Careful about accessibility. Make sure you always test your designs with a keyboard to make sure they work.
If you remove the built in :focus-visible
styles, you must replace them with something equally high-contrast. On the search input you've removed the styling and so there's nothing to indicate that I'm focused on the field.
Make sure to use proper alternative text for images, or an empty string if they are decorative or convey no extra information. An alt text of "avatar" doesn't mean anything to someone viewing the site. If the Github API provides alt text for the user's avatar, then that should be used. If not then it should be an empty string.
Taking a look at your code, you're using a lot of very nested selectors in your CSS. That's going to cause problems with maintainability due to massively increased specificity. Prefer to use single class names and target those. What that means in styled-components is that you'll create individual styled components for the various elements. Rather than nest them all under a single styled component. That way it becomes far easier to override styles when you need to.
Marked as helpful
@shrki416
Posted
@AlexKMarshall Excellent suggestions, thanks so much for taking the time to view my solution and offer feedback, very much appreciated :)