Submitted 12 months ago
Body Mass Index Calculator - Prompt Engineering with ChatGPT
@Shahbaaz92
Design comparison
SolutionDesign
Solution retrospective
The time taken was really long with the logic and converting it into functions. I feel the code is little on confusing side, please overlook for this project.
Community feedback
- @matiasluduena23Posted 12 months ago
Hi SHAHBAAZ ATHHAR’S! Your project work very good and also look nice! Just two advices.
- You can use modular porgramming and use more than one Javascript file. And export functions and classes, this will make you code more clean and easy to debbub. video
- You need a container class in your css to center your content. In the sections where you need to a background image go all away to the margin you can add something like this:
<section > // in this element you background image <div class="container"> your content centered </div> </section> .container { width: 90%; max-width: 1100px; margin-inline: auto; }
Hope this help you!
Good code!
Matias
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin 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