Age-calculator-challenge

Community feedback

• 

  Daniel•330

  @daniel-web-developerPosted over 1 year ago

  Nice one, but I have a couple of suggestions.

  The first suggestion is to add this to the button:

                          cursor: pointer;
  }
  

  It'll transform the cursor from an arrow into that "pointing hand". I think it looks better when it's a button.

  The second suggestion would be to add a JavaScript library to calculate the date for you. I used Datejs. I'm saying this because I've tried using the calculator but it didn't work entirely (I tried '1' in every field and it returned the wrong results). I know it's a really hard algorithm to think by yourself and that's why I used the library.

  Lastly, nice job! Your project looks a lot like the design.

  Marked as helpful

  0

  ---

  

  diana12337•180

  @diana12337Posted over 1 year ago

  I couldn't find the solution for the problem with "1" in the fields so thanks for the Datejs suggestion. I haven't learned about the js libriaries yet so it's a good opportunity to use it.

  Thanks for the feedback!

  0
• 

  Luka Glonti•3,440

  @lack21Posted over 1 year ago

  Great job 👍, but I have a suggestion!

  • Add this to the CSS

  * {
    margin: 0;
  }
  

  this should remove all unnecessary space!

  Marked as helpful

  0

  ---

  

  diana12337•180

  @diana12337Posted over 1 year ago

  Thanks for the suggestion!

  1