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

Freeplay results screen #4105

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Lasercar
Copy link
Contributor

@Lasercar Lasercar commented Feb 5, 2025

Does this PR close any issues? If so, link them below.

#2441

Briefly describe the issue(s) fixed.

Opens a results screen for the current song and difficulty when you press R (currently hardcoded) and it has a score.
I would have it open in a substate and then close it to return to the Freeplay menu so you don't have to reload the freeplay state but doing that seems to cause the freeplay state to become a weird pink screen?

Include any relevant screenshots or videos.

2025-02-05.20-23-23.mp4

@github-actions github-actions bot added status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. pr: haxe PR modifies game code. and removed size: medium A medium pull request with 100 or fewer changes. labels Feb 5, 2025
@Hundrec Hundrec added size: medium A medium pull request with 100 or fewer changes. type: enhancement Involves an enhancement or new feature. status: needs revision Cannot be approved because it is awaiting some work by the contributor. and removed status: pending triage Awaiting review. labels Feb 5, 2025
@Hundrec
Copy link
Collaborator

Hundrec commented Feb 5, 2025

Please base this (and all other feature PRs) on the develop branch.

@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 5, 2025

@Hundrec How do I go about rebasing this branch off the development one?

@Hundrec
Copy link
Collaborator

Hundrec commented Feb 5, 2025

There should be an option at the top of the screen to change the base branch from main to develop!

@Lasercar Lasercar changed the base branch from main to develop February 5, 2025 10:55
@Lasercar Lasercar force-pushed the freeplay-results-screen branch from 83cc662 to 83b30b2 Compare February 5, 2025 10:55
@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 5, 2025

oh cool, that works.

letme go do that for all my PRs.

@Hundrec Hundrec added status: pending triage Awaiting review. and removed status: needs revision Cannot be approved because it is awaiting some work by the contributor. labels Feb 5, 2025
@Hundrec
Copy link
Collaborator

Hundrec commented Feb 5, 2025

Thanks for fixing things up Lasercar!

@Lasercar
Copy link
Contributor Author

Lasercar commented Feb 5, 2025

No problem. Now you can checkout them without issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: haxe PR modifies game code. size: medium A medium pull request with 100 or fewer changes. status: pending triage Awaiting review. type: enhancement Involves an enhancement or new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants