Skip to content
Submitted over 3 years ago

Responsive website with css grid

dkvel•50
@Theguydev
A solution to the Profile card component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Why do I get scroll bars? any suggestions or feedback would be great:)

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • Yazdun•1,310
    @Yazdun

    Hi ! here I have some suggestions

    display:flex is a row by default so you don't need to declare it as a row

    I think this challenge can get done much easier by flexbox rather than grid

    try to learn BEM naming convention ( or any other conventions you prefer ). It will make your code easier to read and maintain.

    also I opened a pull request to your repo which will fix your background and scroll issue

    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