@yanabue
@JohananOppongAmoateng
Submitted
What are you most proud of, and what would you do differently next time?
not really proud could not get the work to center correctly
What challenges did you encounter, and how did you overcome them?
centering elements responsiveness
What specific areas of your project would you like help with?
centering elements responsiveness
@yanabue
Posted
Hi, a few notes to help you out:
When you don't set the body elements height - it's height will only be as high as the content inside of it. If you look at the body element in dev tools - you'll see its only a part of the screens height. So set the body height to 100vh and the card should be centered vertically in the page.
When it comes to the text content inside the card - I suggest you wrap all that content thats not the head image - in a div. Then it would be much easier to manipulate.
Set vertical margin/padding to any elements in the card as needed to space them out from each other.
To set the correct colors in the projects - just use the :root selector to set variable colors, as provided in the styles guide file.
@filipmarcinkowski
Submitted
I thought it would be easier, and maybe that's the reason why I feel like my JavaScript has too much code. I don't know... I hope it's fine :)
I will really appreciate the comments! :)
@yanabue
Posted
Seems pretty good overall! Many error cases included. When entering a date thats in the same year - but still in the future - output is just 0/0/0. I would add this error case to the invalid cases.
Styling wise - looks good although a bit too big on my laptop screen.
Marked as helpful
