Skip to content
Submitted almost 2 years ago

Splitter Tip Calculator

Usama Ahmed•220
@UsamaBinKashif
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Tip calculator app
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Hello, everybody! 👋

This was such a beautiful challenge and I learned so much from making it! I actually wanted to do more, but I felt like it was taking a while and I simply needed to submit my solution. 😅 Of course, feedback is welcome and appreciated! 😊

Happy coding, everyone! 😁

Community feedback

  • Pranshu Sahu•770
    @Pranshu-SahuPosted almost 2 years ago

    congratulations on completing the solution.

    I have few suggestions to improve the solution-

    1. <h1 class="sr-only">splitter</h1> <img src="./images/logo.svg" alt="" /> The rule to follow when using heading tags is to never skip the heading level. In other words, heading levels must always be in order to give structure to a page. *this is would remove your first accessibility error. *

    2. You should use the <main> wrapping entire content for the calculator. *this would remove your second accessibility error `

    I hope this helps. Happy coding! 😄

    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