Skip to content
  • Unlock Pro
  • Log in with GitHub
Profile
OverviewSolutions
7
Comments
4
Aayush Kumar
@AK-CHP1

All comments

  • P
    Cesar Valencia•350
    @cvalencia1991
    Submitted over 2 years ago

    Preview Card Component

    #accessibility#airtable
    3
    Aayush Kumar•110
    @AK-CHP1
    Posted over 2 years ago

    Hi @cvalencia1991! Congratulations on completing this project. I think you can further improve the typography of your paragraph by adding line-height: 1.5. Also consider using CSS grids to properly leave gaps around your cards. Your .stylenumber element also seems larger than the card's heading, consider making it quite smaller.

    Rest looks great!

    Marked as helpful
  • Gobinath•140
    @gopinath-97
    Submitted almost 3 years ago

    Product Card

    3
    Aayush Kumar•110
    @AK-CHP1
    Posted almost 3 years ago

    Congratulations on completing this project

    Your solution looks good. But I think you can further improve this solution by;

    • Adding a line-height of 1.5 to your paragraphs
    • Using flexbox in your .content element for proper alignment and spacing of different components.

    Happy coding!

  • TEKE•30
    @Teke111
    Submitted almost 3 years ago

    QR scan page using semantic HTML5 markup and CSS properties

    2
    Aayush Kumar•110
    @AK-CHP1
    Posted almost 3 years ago

    Congratulations on completing this project

    I think you can further improve your project by using media queries to change layout for mobile devices (max-width: 375px as per the style guide)

    Also you can learn the basics of flexbox at:

    • MDN here
    • Wes Bos's free course at flexbox.io

    Please consider checking my solution here and provide some feedback.

    Please let me know it that helped.

    Thanks

  • Élder Ramos•30
    @elder-ramos
    Submitted almost 3 years ago

    QR Code Component

    2
    Aayush Kumar•110
    @AK-CHP1
    Posted almost 3 years ago

    Congratulations on completing the project

    I think the project would look better if you vertically center the .card section element. You can do that either by playing with margin values or using something like flexbox or positions (margin: auto doesn't help in centering vertically, you could instead use margin: <some value> auto ) . Also consider setting font-size of p element to 15px (as mentioned in the style guide), because it seems too small and almost becomes unreadable on small viewport. You should also use media queries to make some changes with the card for mobile layout.

    All the best. Please consider checking my solution of this problem here

    Any feedback and suggestions on how I can improve are very welcome!

    Marked as helpful

Stay up to datewith new challenges, featured solutions, selected articles, and our latest news

Frontend Mentor

  • Unlock Pro
  • Contact us
  • FAQs
  • Become a partner

Explore

  • Learning paths
  • Challenges
  • Solutions
  • Articles

Community

  • Discord
  • Guidelines

For companies

  • Hire developers
  • Train developers
© Frontend Mentor 2019 - 2025
  • Terms
  • Cookie Policy
  • Privacy Policy
  • License

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub