
Design comparison
Solution retrospective
My first Sass solution. I really love Sass. Also I know it says grid but I used flexbox instead.
Community feedback
- @Sk7867Posted over 3 years ago
Hi Michelle, It looks good on desktop but in mobile view and tablet view, 3rd section is not visible, in
body
putmin-height: 100vh
instead ofheight: 100vh
andpadding-top: 20px
or any value you like so it doesn't stick to the top in mobile view.Marked as helpful1@mcdulingmPosted over 3 years ago@Sk7867 thanks for the feedback. I just tested on some of the other mobile options and I see what you mean. Going to add it.
0 - P@palgrammingPosted over 3 years ago
Looks nice but you should put a
:hover
state on your buttonMarked as helpful1@mcdulingmPosted over 3 years ago@palgramming ah thanks! I worked on it over a month ago and just posted without checking.
0@mcdulingmPosted over 3 years ago@palgramming I see you have a lot of experience. Do you have any advice on the accessibility issues it's presenting within this solution?
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