Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @indigorose

    Submitted

    What are you most proud of, and what would you do differently next time?

    This challenge extended my knowledge of flexbox. The initial mobile or small screen styling kept the flexbox of the four cards simple at

    .container {
      display: flex;
      flex-direction: column;
    } 
    

    When working on the desktop version, the middle cards were placed within a wrapper to maintain the column flex direction with the container moving the flex-direction: row; layout.

    What challenges did you encounter, and how did you overcome them?

    No major challenges for this solution as it was small with only four cards. If the layout was say the bento style or contained more cards, then it would be more of a challenge as I do not work with CCS grid often.

    What specific areas of your project would you like help with?

    No particular areas at the moment, if a major refactor is required, it maybe in finding a solution to have the middle cards in the column format without the need for a separate 'wrapper' around them.

    @Saritha-Sasi

    Posted

    I've just completed a front-end coding challenge from @frontendmentor! 🎉 Any suggestions on how I can improve are welcome!

    0
  • @Saritha-Sasi

    Posted

    Your HTML and CSS look well-structured, but here are a few potential improvements or minor issues:

    Duplicate <link rel="preconnect"> for Google Fonts: You have duplicated the <link rel="preconnect"> for both Google Fonts and Google Fonts Static. It is enough to include these only once:

    <link rel="preconnect" href="https://fonts.googleapis.com" /> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin />
    0