Design comparison
Solution retrospective
updated version. would you say its much better and where can i improve on it. also i really need help with the responsive side of this. thanks in advance.
Community feedback
- @graynnejiPosted over 1 year ago
Another simple way to make it responsive without using media query.. Lets say you use grid for the main-container
.category-container { display: grid; grid-template-columns: repeat(auto-fit, minmax(20rem, 1fr)); place-items: center text-align: center; }
Grid columns breaks after hitting 20rem of width. I hope this helps you in your future projects.
Marked as helpful1@xdcronPosted over 1 year ago@graynneji oh wow i didnt know this will definetetly apply in the future
0 - @0xabdulkhaliqPosted over 1 year ago
Hello there π. Congratulations on successfully completing the challenge! π
- I have other recommendations regarding your code that I believe will be of great interest to you.
POINTING CURSOR βοΈ:
- Looks like the component's
button
element has not a pointer, this property plays a major-role in terms of both UI & UX
- The
cursor: pointer
CSS property is important for button-like elements because it changes the cursor from the default arrow to a pointer when hovering over the element. This provides a visual cue to the user that the element is clickable and encourages interaction.
- In terms of UI/UX, using
cursor: pointer
helps to improve the usability of the interface by making it easier for users to identify interactive elements. It also helps to provide feedback to the user by indicating which elements are clickable and which are not.
- So we want to add this property to the following
button
element
button { cursor: pointer; }
- Now your component's
button
has got the pointer & you learned about this property as well
.
I hope you find this helpful π Above all, the solution you submitted is great !
Happy coding!
Marked as helpful1@xdcronPosted over 1 year ago@0xAbdulKhalid Noted will apply it too all my buttons movings foward
0 - @Kamlesh0007Posted over 1 year ago
Great job on writing this code! You have demonstrated a good understanding of the concepts and applied them effectively to solve the problem. One suggestion I have is to add this code to make main-container with box shadow and border radius
.main-container { font-size: 1rem; display: flex; align-items: center; flex-wrap: wrap; box-shadow: 0.5rem 1rem 3rem hsl(224, 30%, 27%, .2); border-radius: 2rem; }
and for responsive design make main-container with flex direction:column and remove the max width
Marked as helpful1@xdcronPosted over 1 year ago@Kamlesh0007 thank you i really wasnt sure on how to add the box shadow
0@Kamlesh0007Posted over 1 year ago@xdcron good one Buddy but u have to add border radius for result container towards bottom
0
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