Skip to content

fix: correct drag coefficient label and units in ParabolicMotion#309

Merged
mattqdev merged 1 commit into
physicshub:mainfrom
TechGenius-Karan:fix/drag-label-units
May 9, 2026
Merged

fix: correct drag coefficient label and units in ParabolicMotion#309
mattqdev merged 1 commit into
physicshub:mainfrom
TechGenius-Karan:fix/drag-label-units

Conversation

@TechGenius-Karan
Copy link
Copy Markdown
Contributor

📘 Pull Request Template – PhysicsHub

Thank you for contributing to PhysicsHub!
Please complete the sections below to help us review your pull request efficiently.


🔍 Description

The drag input was labeled "Linear drag (1/s)" but the simulation calls
airResistance() with quadratic drag. This mislabels the physical quantity and gives
students the wrong units. Updated the label to "Quadratic drag (kg/m)" to match the
actual computation.

Closes #BUG-010 (if applicable)


✅ Checklist

Before requesting a review, please ensure that you have:

  • Verified that the project builds and runs locally (npm run dev)
  • Ensured no ESLint or TypeScript warnings/errors remain
  • Updated documentation, comments, or in-code explanations where needed
  • Verified responsiveness across devices (desktop, tablet, mobile)
  • Followed the CONTRIBUTING.md guidelines

🎨 Visual Changes (if UI-related)

IF CHANGES ARE RELATED TO SIMULATIONS PLEASE SEND A SHORT CLIP ABOUT IT
(OBLIGATORY)


📂 Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 📝 Documentation update
  • ♻️ Refactor / code quality improvement
  • 🎨 UI/UX enhancement
  • 🔒 Security improvement

🧩 Additional Notes for Reviewers

@mattqdev mattqdev merged commit cfe1011 into physicshub:main May 9, 2026
1 check passed
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.

2 participants