Design comparison
Solution retrospective
this is missing footer display div three space between problem
okay feedback working
Community feedback
- @VenusYPosted 9 months ago
Good job on this challenge!
There are a few minor changes you could make to make it even better though.
I noticed that the buttons don't change when being hovered over other than the cursor changing to a pointer.
While this technically does still notify the user that the button is a clickable object, it's not very good for user experience as it's not a drastic change.
I would recommend you add some extra styles to the button when being hovered over, focused on, and clicked for an improved user experience.
Some examples of changes you could make are the background colour, button size, font colour, etc. All these things could make it much more obvious that the user is interacting with the button.
I also saw that the 'Grow Together', 'Flowing Conversations', and 'Your Users' sections were not centered properly.
You can fix this by using flexbox like so:
.container { display: flex; flex-direction: column; align-items: center; } .nav { width: 100% }
By using flexbox to center the content of the page, it causes the nav bar to be squished due to
align-items: center
.A simple fix for this is to set
width: 100%
on the.nav
element. This allows it to take up all the available horizontal space within the flexbox.Other than that, this is a very good solution!
I hope this has been helpful! :)
Marked as helpful0@578shivamPosted 9 months agohello ,, now update hover button enough ... this is fix not working nav problem okay
bro thank @VenusY
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