@sikulamartin
Posted
on widther screens and on mobile it looks terrible, thats because you used display flex propety wrong, also the section.calculator into looks terrible because of the wrong usage of display:flex i would advise you look at some youtube tutorial or just go back to the learning path about the layouts, your js LGTM, and one last thing that i noticed, on the submit button there should have been linear gradient insted of just a color, so if you dont know how to do that, theres a site, that does it for you : https://cssgradient.io/
Marked as helpful
@A-noob-in-Coding
Posted
@sikulamartin Thanks for the constructive feedback, I'm gonna Incorporate the suggestions
@A-noob-in-Coding
Posted
@sikulamartin Hi there can you review the site again, I would love your feedback
@sikulamartin
Posted
@A-noob-in-Coding Hi there! I checked out the site, and it’s definitely looking better. I do have a few suggestions, though:
On line 180 of your styles.css, you’ve set the image height to 60vh. It might be better to set it to auto instead. For lines 198 and 199, I'd recommend setting both to auto as well. There’s no need to make them bigger than necessary. I’d suggest removing line 197. Without padding on top, it looks a bit off. That’s it for the first page. For the success message, I’d recommend giving the entire .success-container a width of around 375px, and setting the height to auto within your media query.
I hope these suggestions make sense! ;)