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

Age Calculator

Djonali 110

@Djonali

Desktop design screenshot for the Age calculator app coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi, Please review my solution and provide your feedback.

Community feedback

Basha 330

@Bashamega

Posted

Hello Djonali:)

I have tested your project multiple of times. I have noticed only three issues.

  1. I have noticed that you can enter letters to the input, but the user should be able to enter only number. You can fix it by adding the type arttibute This is an example ob how you can fix it
<input type="text" id="days" class="form-container-input" placeholder="DD" oninput="onInputChange(days.value, event)" type="number">
  1. I have noticed that when you hover on the button it doesn't make the cursor a pointer so the user don't know where to press. You can add the cursor property to the CSS
  2. I think you should replace the form-container div to a form Other than that your project is great

Marked as helpful

1

Djonali 110

@Djonali

Posted

@Bashamega , Many thanks for the feedback. I will incorporate the suggested changes and submit the code.

0
Basha 330

@Bashamega

Posted

@Djonali Your welcome

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