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

[BUGFIX] Fix Input Offset interactions with Input Window (+ sustains) #3708

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

Conversation

xenkap
Copy link

@xenkap xenkap commented Oct 15, 2024

Very odd that this was already done on the opponent's side, but not the Player's.

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

#3692, #2382

Briefly describe the issue(s) fixed.

Input Windows would not be offset by Input Offset, only Note Judgement would be. This pull request fixes that.

Include any relevant screenshots or videos.

BEFORE:

2024-10-15.20-22-12.mp4

AFTER:

2024-10-15.20-19-25.mp4

(Recorded on 200ms Input Offset)
(side note: can someone make events responsive to your offset ?? the hey animations are playing too early) ignore, use visual offset on negative side to achieve effect

yuppe
isn't it odd that opponent windows are already accounted for Like
@github-actions github-actions bot added medium A medium pull request with 100 or fewer changes haxe Issue/PR modifies game code labels Oct 15, 2024
@xenkap
Copy link
Author

xenkap commented Oct 15, 2024

oops wait wrong implementation? hold on

@xenkap xenkap marked this pull request as draft October 15, 2024 11:39
@xenkap xenkap marked this pull request as ready for review October 15, 2024 11:45
@xenkap
Copy link
Author

xenkap commented Oct 15, 2024

my bad, completely correct implementation i just forgot to set my binds

@xenkap
Copy link
Author

xenkap commented Oct 15, 2024

All good, ready for review :D

@xenkap xenkap changed the title Fix Input Offset interactions with Input Window (+ sustains) [BUGFIX] Fix Input Offset interactions with Input Window (+ sustains) Oct 15, 2024
@swordcube
Copy link

someone please accept this 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
haxe Issue/PR modifies game code medium A medium pull request with 100 or fewer changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants