Skip to content
OverviewSolutions
9
Comments
4
Khael
@i-am-Khael

All comments

  • Daniel•30
    @Aflora22Submitted over 2 years ago

    Well... now I understand why I need to think before start coding something... this was beyond my knowledge :(

    NFT page using HTML and CSS

    3
    Khael•230
    @i-am-KhaelPosted over 2 years ago

    Your solution seems OK. Just keep on grinding.

    1
  • Khael•230
    @i-am-KhaelPosted over 2 years ago

    Note to everyone who will stumble upon my Solution, the API key I'm using reach its 1000 request limit since I'm using the Free Subscription..

    0
  • Khael•230
    @i-am-KhaelPosted over 2 years ago

    Great effort..

    You can use this API endpoint https://api.adviceslip.com/advice/{id}, and generate a random number to replace id.. so that every time a user clicks on dice icon it will generate a random advice base on the id.. You can refer to my solution, https://www.frontendmentor.io/solutions/advice-generator-app-using-html5-css3-and-vanilla-js-S1-ZYT7Q5

    Marked as helpful
    1
  • Khael•230
    @i-am-KhaelPosted over 2 years ago

    To resolve those accessibility issue,

    • use an h1 instead of h3
    • use main for the <main></main> landmark, then <section></section> to set landmark, then use div(s) inside section element.

    and you can change api endpoint to https:// api.adviceslip.com/advice/id , then using Math.floor(Math.random * 255) to generate a random number for the id.

    Great work and effort, though.

    Marked as helpful
    0