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

Fix cursor rotating with barrel roll mod active #30326

Merged
merged 2 commits into from
Oct 21, 2024

Conversation

jhk2601
Copy link
Contributor

@jhk2601 jhk2601 commented Oct 17, 2024

Closes #28693

Some may argue that cursor rotation is intended behavior but it looks odd and feels pretty janky, especially on abnormal cursors like the issue describes. Would be better if cursor rotation could be adjusted in existing switch statement but didn't want to mess with existing implementation to allow that and this way is simple and readable as well.

@bdach
Copy link
Collaborator

bdach commented Oct 18, 2024

Honestly not sure what to think about this one in general. It feels to me that this being an issue that needs to be fixed is disputable in the first place. The cursor is on the playfield, the playfield is rotating - so why wouldn't the cursor rotate too?

The only reason I can conjure up to support this change is that skins with non-symmetrical cursors are affected more visually by this. But I honestly see the skin linked in the issue as a "meme skin" (it's trying to fake the player playing CS, for crying out loud), so I'm not sure it's serious enough to warrant adding carve-outs of this sort.

@ppy/team-client one of low priority I'd like to gather opinions about

@smoogipoo
Copy link
Contributor

smoogipoo commented Oct 18, 2024

Hopefully we share the opinion that we should support all skins that are non-abusive to the best of our ability? Many are made just for fun and I don't want to focus on the meme status when deciding to support them. I see that skin as less abusive than any of the more modern skins that we've supported...

However, I don't think this has anything to do with the skin and is more about gameplay mechanics. I'd wager that when using barrel roll you kinda know what you're getting yourself into?

So although I'm 50/50 on this, I'm tending towards not applying the change.

@peppy
Copy link
Member

peppy commented Oct 18, 2024

I think it makes sense to not rotate the cursor with the playfield. To me the concept of barrel roll is that the hitobjects are rotating. As a player I wouldn't expect it to affect the cursor.

@Susko3
Copy link
Member

Susko3 commented Oct 18, 2024

As a player I wouldn't expect it to affect the cursor.

Yeah, since the movement of the cursor is unaffected by barel roll, its rotation should also be unaffected.

For example, if using Windosu XP (skin of the year 2023 runner-up) I would expect the "pointer" cursor to always point up.

@jhk2601
Copy link
Contributor Author

jhk2601 commented Oct 18, 2024

As a player I wouldn't expect it to affect the cursor.

Yeah, since the movement of the cursor is unaffected by barel roll, its rotation should also be unaffected.

For example, if using Windosu XP (skin of the year 2023 runner-up) I would expect the "pointer" cursor to always point up.

Yeah this is a great example, considering that almost all cursors are symmetrical anyway we are only really dealing with edge cases like this here, and in those edge cases it is more logical and expected for the cursor to stay aligned. (also worth noting that it will feel jarring because the cursor doesn't APPEAR to be rotating when using a "normal" cursor, so it will feel like a setting was changed or something)

@peppy peppy added the next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! label Oct 21, 2024
@bdach bdach self-requested a review October 21, 2024 10:13
@bdach bdach changed the title Cursor no longer rotates when using Barrel Roll mod in standard Fix cursor rotating with barrel roll mod active Oct 21, 2024
Copy link
Collaborator

@bdach bdach left a comment

Choose a reason for hiding this comment

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

I guess this does what it says

@bdach bdach merged commit 22c3a25 into ppy:master Oct 21, 2024
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:mods area:skinning next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! ruleset/osu! size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enabling the mod Barrel Roll will cause the cursor to rotate with the game field
5 participants