Skip to content
Submitted over 3 years ago

Advice-Generator

Shonuff•280
@TheShonuff
A solution to the Advice generator app challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

After being away for a few years, this is my first challenge project. I'm happy that it's functioning as intended but I've seen my CSS break in a few cases. I don't think I setup my flex model correctly to account for the longer quotes. If you have the time to supply any type of feedback it would be greatly appreciated. Thank you.

Code

Please log in to post a comment

Log in

Community feedback

  • Account deleted

    Hello there! 👋

    Congratulations on finishing your challenge! 🎉

    I have some feedback on this solution:

    • put the quote inside a <blockquote> tag instead of p to be more semantic

    • check the HTML report to fix accessiblity issues

    • add cursor pointer to button container so that the user knows its clickable

    i hope this is helpful and goodluck!

    Marked as helpful
  • Евгений•50
    @rahunak

    thanks for this code : {cache: 'no-store'} it decided my problem

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