Skip to content
Submitted over 3 years ago

Responsive page using css grid

Vishal Kumar•210
@LahsivK4070
A solution to the Four card feature section challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

I feels like i did a pretty great job. If i am missing somewhere do let me know. Thanks

Code

Please log in to post a comment

Log in with GitHub

Community feedback

  • Yazdun•1,310
    @Yazdun

    Hello Vishal 👋 I have one thing to add to argel's great feedback !

    instead of using p tag for your headings, use h1 to fix your ACCESSIBILITY issue

    Marked as helpful
  • argel omnes•1,800
    @argelomnes

    Hey Vishal,

    Nice work with the layout. For the "missing somewhere", that's probably:

    1. font-sizes
    2. 4th card should be labeled as 'Calculator'
    3. horizontal spacing of the cards is too wide

    Bonus: since you're using Grid, you can remove card-2 and card-3 from their container and position each where they should be.

    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