Skip to content
Submitted over 2 years ago

Advice generator -responsive layout

pranavā€¢120
@PrAnAvViNaYaKjAdHaV
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Advice generator app
  • HTML
  • CSS
  • JS
  • API
2junior

Design comparison

SolutionDesign

Solution retrospective

I think that button is still not in the center if you can help then, please comment it's helped me šŸ˜ƒšŸ˜ƒšŸ˜ƒšŸ˜ƒšŸ˜ƒ

Community feedback

  • Shashree Samuelā€¢9,260
    @shashreesamuelPosted over 2 years ago

    Hey Pranav, good job completing this challenge. Keep up the good work

    Your solution looks great however I think your card dice button needs some margin from the top using the margin-top property in addition the green dice needs some padding using padding

    In terms of the accessibility issue simply wrap all your content between <main> tags

    I hope this helps

    Cheers Happy coding šŸ‘

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

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