Skip to content
Submitted over 1 year ago

Tip calculator

itsCoolguy•180
@itsCoolguy
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

Some things that I know are 'bad' about the project I have made:

  • In the JavaScript, in the 'on' function parts, I have repeated basically the same block of code 3 times, which is probably quite inefficient
  • There is a lot of things that could be optimised about the CSS, such as combining a few things into one

Any feedback on this project would be greatly appreicated!

Community feedback

  • C H A R A N 🎯•2,670
    @0xabdulPosted over 1 year ago

    Hey there ! 👋 Congratulations you finished the Tip Calculator App...🎉

    • some suggestions for you improve your code

    Html 🏷️:

    • for Accessibility reports you using the and div tag to wrap the whole html code and put attribute role="main" is correct way 🎯 and using the header tag those all are problem to occur the Accessibility reports for example...
    <body>
<div class="container" role="main">

//Whole html code

<h1> </h1>
<h2> </h2>
<h3> </h3>
//by ordering using the header tag

</div>
</body>
    • I Hope it's useful for you and wating for your next project ❤️
    • 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