Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Advice Generator App Using Tailwind CSS

#tailwind-css
harshj11 50

@harshj11

Desktop design screenshot for the Advice generator app coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
  • API
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any suggestions are welcomed! Please do share your reviews, Thanks :)

Community feedback

Ahmed Bayoumi 6,800

@Bayoumi-dev

Posted

Hey! Congratulations on completing this challenge... You have accessibility issues that need to fix.

  • Buttons must have the discernible text, So don't leave the alt attribute empty when using the image <img> inside the button <button> without text.
<button class="absolute left-2/4 bg-neongreen p-4 rounded-[50%] -translate-x-1/2 hover:shadow-neongreen" id="generate-advice">
        <img src="./images/icon-dice.svg" alt="Advice Generator">
      </button>

Or set the attribute aria-label to describe the button.

  • Page should contain a level-one heading, Change <div class="text-xs text-center uppercase text-neongreen text-[10px] tracking-[.25em]"> to <h1 class="text-xs text-center uppercase text-neongreen text-[10px] tracking-[.25em]"> You should always have one h1 per page of the document... in this challenge, you will use h1 just to avoid the accessibility issue that appears in the challenge report... but don't use h1 on small components <h1> should represent the main heading for the whole page, and for the best practice use only one <h1> per page.
  • An img element must have an alt attribute, to provide alternative information for an image if a user for some reason cannot view it, but in this case, add an empty alt attribute to avoid the accessibility issue because it is an unnecessary icon for the content.
<img src="./images/pattern-divider-desktop.svg" alt="" class="hidden md:block">

I hope this is helpful to you... Keep coding👍

1

harshj11 50

@harshj11

Posted

@Bayoumi-dev thank you so much for the suggestions I would surely work on them.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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