SCSS grid and flexbox layout

Solution retrospective
Any feedback is welcome , Thank you !
Please log in to post a comment
Log in with GitHubCommunity feedback
- @mjbaga
Hi Ali. Nice work on the challenge.
Just some feedback:
- Was already pointed out on the social media icons. This usually happens when height and width doesn't match. You can try setting an explicit height and width, maybe in pixels. Or you can define just the width or height and use
aspect-ratio: 1/1;
- I'm on a larger screen and the social media icons are cut off, almost by half and can't scroll down. Maybe doesn't need the
overflow: hidden
on body. - On your button, maybe you can add a small transition effect,
transition: color 300ms ease-in;
. These are micro-interactions that improve user experience.
Hope this helps! Happy coding!
Marked as helpful - Was already pointed out on the social media icons. This usually happens when height and width doesn't match. You can try setting an explicit height and width, maybe in pixels. Or you can define just the width or height and use
- @Deevyn9
Wow, i almost couldn’t differentiate the one you made from the original, there’s a very minor issue though, the social media icons have an oval border, you can make it fully round by setting the border radius to 50%
Marked as helpful
Join 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