Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BMICalculator/DevanshKyada27 #1010

Closed

Conversation

DevanshKyada27
Copy link
Contributor

Fixes #806

Created a BMI Calculator using HTML, CSS and JS.

Hope this solves the issue.

Attached the screen shots and the screen recording of the project below.
Please do not consider the script.js file.

Looking forward for merging this PR.

Developer Checklist

  • [ ✅] Followed guidelines mentioned in the readme file.
  • [ ✅] Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
  • [ ✅] Starred ⭐ the Repo (Optional)

Summary

Created a BMI calculator using HTML, CSS and JS which takes age height and weight as input and gives the BMI as output.

Screenshot

image

Live Project Link

127.0.0.1_5500_index.html.-.Personal.-.Microsoft.Edge.2023-10-15.17-54-17.mp4

Copy link
Collaborator

@NitkarshChourasia NitkarshChourasia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything is good.
Add a README.md file, as mentioned in the repository's README.md file.

@DevanshKyada27
Copy link
Contributor Author

Hey @NitkarshChourasia can I make a new PR and add all the necessary files.

@NitkarshChourasia
Copy link
Collaborator

Just deliver me the result, Rest I don;t care how you do it.

@DevanshKyada27
Copy link
Contributor Author

Sure @NitkarshChourasia will make it soon :)

@DevanshKyada27
Copy link
Contributor Author

Hey @NitkarshChourasia created a new PR with all the changes.
Hope you like it.
Looking forward for merging the PR.

@NitkarshChourasia
Copy link
Collaborator

README.md file with assets embedded?

@DevanshKyada27
Copy link
Contributor Author

Hey @NitkarshChourasia are you talking for the PR #1029 ?

@NitkarshChourasia
Copy link
Collaborator

Please resolve the conflict !

@thinkswell
Copy link
Owner

Hi @DevanshKyada27,
Both of your projects are amazing! 👍.
Please follow proper directory guidelines mentioned in the README file.

Thank you.

Copy link
Collaborator

@NitkarshChourasia NitkarshChourasia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks all good. Let's see.

@NitkarshChourasia NitkarshChourasia self-requested a review May 7, 2024 10:02
@NitkarshChourasia
Copy link
Collaborator

This branch has conflicts that must be resolved
to resolve conflicts before continuing.
Conflicting files
index.html
script.js
style.css

@DevanshKyada27

Please resolve this conflict or with time I would have to close this issue.
Thank you.
@NitkarshChourasia

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a BMI calculator
3 participants