-
-
Notifications
You must be signed in to change notification settings - Fork 42
Test Simulation #120 Done #179
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
base: main
Are you sure you want to change the base?
Conversation
|
Hey, your PR seems very interesting, but It's too complex to understand through screenshots. |
|
And there is also a conflict with package-lock.json |
|
Hello @physicshub, I have made the changes and resolved the conflicts as well. The change can be merged properly now. Thanks for letting me know. |
|
Hey sorry for late reply, there is something else that you should change:
The system that you made is beautiful, I love the idea of pre-made templates that you can apply, would be awesome one day to create these template also for more sims. |
|
Hey Matt, apologies for the errors found. I will fix these errors today and update the PR for this issue. Thank you for bringing this problem into account so, I can work and improve myself by working into this. |
|
Hey Matt, I have done the changes. I think you can commit the changes now. Thanks again for letting me know my mistakes. |
|
Thank you, can I see how it looks now the page? |
|
So, it looks similar but, I have done the formatting issues and also fixed the issues with the package.json file. I have kept the styling of the test simulation similar as it is a test simulation, so I have kept it little bit differently so that it will be easier to update or work on in the future. Also, I was trying to fix the lint and format checks but, it is not getting fixed. I will continue to fix that though. |
|
Thank you for fixning this. Sorry for the delay, I'm very busy with Erasmus+ program at the moment. I'll see your changes at the end of the week when my french twin will go away |
|
Sure that's not an issue. Thank you. |
π Pull Request Template β PhysicsHub
Thank you for contributing to PhysicsHub!
Please complete the sections below to help us review your pull request efficiently.
π Description
This PR implements comprehensive improvements to the Test Simulation (#120), focusing on UI/UX enhancements, educational features, and bug fixes.
π― Main Changes
1. UI/UX Improvements
2. New Educational Components
CollapsibleSection.tsx- Reusable collapsible section component for organizing educational contentGuidedExperiments.tsx- Component for displaying and applying predefined experimentsLearningObjectives.tsx- Component for displaying learning goals and key variablesPhysicsEquations.tsx- Component for displaying relevant physics equationsPhysicsWarnings.tsx- Component for displaying physics-related warningsEducationalTheorySection.tsx- Comprehensive educational content section3. Bug Fixes
4. Code Quality
π§ Technical Details
πΈ Visual Changes
β Testing
π Educational Impact
This PR enhances the educational value of the Test Simulation by:
Related Issue: Test Simulation #120
Type: Enhancement / Bug Fix / UI/UX Improvement
Closes #120 (if applicable)
β Checklist
Before requesting a review, please ensure that you have:
npm run dev)π¨ Visual Changes (if UI-related)
PhysicsHub - Test for benchmarks - Google Chrome 2026-02-02 16-22-07.zip
π Type of Change
π§© Additional Notes for Reviewers