Skip to content
OverviewSolutions
16
Comments
2
Ezekiel Abolade
@thefacecodes

All comments

  • Ezekiel Abolade•360
    @thefacecodesPosted over 2 years ago

    Hi Isaac, your work looks great, but you appear to have accessibility issues.

    Wrapping your content in between <main>....</main> should help fix this issue.

    Your project does not fetch new advice on click of the button too, I faced this issue too, appending t=" + Math.random() to the API fetch link worked for me, you can try it out.

    Great project.

    1
  • Ezekiel Abolade•360
    @thefacecodesPosted over 2 years ago

    Hi Ranjana, Your work looks great, but I have a little suggestion on the responsiveneess,

    Trading the "max-width: 20%" on the card element to a fixed width between 300px to 350px might help with the responsiveness.

    0