Submitted about 3 years ago
GitHub User Search app with Sass, JS, CSS Grid, Flexbox and Parcel
@dmitrymitenkoff
Design comparison
SolutionDesign
Solution retrospective
This was the most challenging app I've completed to-date. I'm really keen to learn more about theme changing.
Cheers!
Community feedback
- @Dharmik48Posted about 3 years ago
Hey👋,
Good job with the solution! But there a some issues you can fix:
- Maybe remove the default outline from
input
andbutton
when focused. - And when the page first loads the data is empty so maybe you can add a default profile of someone which is there on first load
Apart from these it is really good, keep it up👍
Marked as helpful3@dmitrymitenkoffPosted about 3 years ago@Dharmik48 Thanks for your feedback. I've added the focus styles manually (autofocus on input) for accessibility reasons, although i realise these are not part of the provided design files. I'll play with these styles more and see if any other outline styles might look better! Cheers!
0 - Maybe remove the default outline from
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord