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 App | Mobile First | Flexbox

@faruquedewan

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


This was a chanllenge I enjoyed a lot. Any feedback on parts where I can improve will be appreciated. Thank you.

Community feedback

@0xabdulkhaliq

Posted

Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

CSS 🎨:

  • Looks like the component has not been centered properly. So let me explain, How you can easily center the component without using margin or padding.
  • We don't need to use margin and padding to center the component both horizontally & vertically. Because using margin or padding will not dynamical centers our component at all states
  • You already using Grid for layout, but you didn't utilized it's full potential. Just add the following rules to properly center the component.
body {
    place-items: center;
}
  • Now remove these styles, after removing you can able to see the changes
main {
  margin: 20% auto;
}
  • Now your component has been properly centered

.

I hope you find this helpful 😄 Above all, the solution you submitted is great !

Happy coding!

Marked as helpful

0

@faruquedewan

Posted

@0xAbdulKhalid I just realized I used align-content instead of place-content. I just tried place-items after going through your suggestion and it looks good but has a slightly bigger width. Can you please elaborate on the difference between place-content and place-items?

1

@Sinkycode

Posted

Congratulations 🎉👏 mate, it was a neat work. My observation -

  • Your validation was not really friendly (like making sure users understood where they missed it and fixing it immediately )
  • Your validation only shows one at a time (I feel you tried to fix it but that's all you could do for you not to mess the whole code up )

Congratulations 👏🎉 once again. I really love what frontend mentor is doing here. I'm learning a lot from people's project

Marked as helpful

0

@faruquedewan

Posted

@mosfresh Yeah, my JavaScript skills seriously needs a lot of work as of now. All I was trying to to was make it work and not mess up the whole code.

1

@Sinkycode

Posted

@faruquedewan Smiles 😊 I believe you'll get better, it's just a matter of time

1

@faruquedewan

Posted

@mosfresh Hey, check my new solution. I have updated the code. :)

1

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