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 UI fixes #4214

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

Conversation

Lasercar
Copy link
Contributor

@Lasercar Lasercar commented Feb 23, 2025

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

Fixes #3190, fixes #3627, fixes #4037, fixes #3703

Briefly describe the issue(s) fixed.

This PR fixes four issues - The letters not looping their animation and restarting it when selected, the lettersort input not being disabled when a capsule is opened, the first rank of a song overriding itself, and the instrument selector using the capsule's current position, rather than the position the capsule is targeting.

Also, if the capsule has a null target song or difficulty, it now also re-enables input before returning.

Another thing I noticed while doing this - only the random capsule does it's jump in, the rest of the capsules don't. I tried making the rest of the capsules jump in and I can see why they currently don't - they don't move very smoothly to their positions when they jump in and it looks really obvious. Setting the value to the targetPos.x rather than the capsule's x position fixes it, but then the capsules aren't set to their proper positions until you change selection. I'll figure out a fix proper fix for this later.

Also partially fixes #2659

Include any relevant screenshots or videos.

old video with fixes for the first two issues
2025-02-23.19-13-38.mp4
2025-02-24.22-21-05.mp4
2025-02-26.00-32-19.mp4

Input is disabled as soon as possible when a capsule is open
Input re-enables after capsule errors
instrument selector uses the capsule's target position as it's position, rather than the capsule's position
@Lasercar Lasercar changed the base branch from main to develop February 23, 2025 09:26
@github-actions github-actions bot added pr: haxe PR modifies game code. status: pending triage Awaiting review. size: medium A medium pull request with 100 or fewer changes. labels Feb 23, 2025
@AbnormalPoof AbnormalPoof added type: minor bug Involves a minor bug or issue. topic: polish Involves minor polish to the UI or gameplay. labels Feb 23, 2025
@Lasercar Lasercar changed the base branch from develop to main February 24, 2025 12:25
@Lasercar Lasercar changed the base branch from main to develop February 24, 2025 12:25
@Lasercar Lasercar changed the base branch from develop to main February 24, 2025 12:25
@Lasercar Lasercar changed the base branch from main to develop February 24, 2025 12:26
Copy link

@JackXson-Real JackXson-Real left a comment

Choose a reason for hiding this comment

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

Looks like most of the fixes work fine. There is one problem I've noticed though regarding the squished capsules. Whatever you did seems to have fixed it mostly as I don't see it anymore aside from one very specific circumstance. Here is a video of it happening:

Base.Profile.2025.02.24.-.20.27.44.18.mp4

Doing these exact steps should reproduce the bug consistently. Since this seems to be linked to the issue of normal songs being shown in freeplay after exiting an erect song, I think it's safe to assume that your PR #4088 will get rid of the issue. I'm going to approve this PR since it fixes exactly what it says it does, and once your other PR is merged, the squished capsule bug should be completely fixed.

Edit: This actually happens whenever you exit out of any song, and it's always Darnell BF Mix, I haven't seen it happen to any other song.

@Lasercar Lasercar changed the base branch from develop to main February 25, 2025 14:42
@Lasercar Lasercar changed the base branch from main to develop February 25, 2025 14:42
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. topic: polish Involves minor polish to the UI or gameplay. type: minor bug Involves a minor bug or issue.
Projects
None yet
3 participants