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

Add a toggle for star fountains during gameplay #30894

Merged
merged 6 commits into from
Nov 28, 2024

Conversation

HenintsoaSky
Copy link
Contributor

@HenintsoaSky HenintsoaSky commented Nov 27, 2024

Adds an option under Gameplay > General Settings that allows users to enable/disable star fountains during gameplay.

image

@frenzibyte
Copy link
Member

Thanks for the contribution!

We generally avoid adding new options to the settings without a convincing reason to do so, has there been any discussion elsewhere prior to the submission of this PR?

@HenintsoaSky
Copy link
Contributor Author

In : #29830

@peppy
Copy link
Member

peppy commented Nov 27, 2024

For added context, this was done until now by skins making transparent images for the star sprite. Which isn't optimal as it means stars disappear from everywhere in the game.

Not strongly against this one existing.

@Joehuu
Copy link
Member

Joehuu commented Nov 27, 2024

Just noting there was an alternative solution in #29792 (comment).

@frenzibyte
Copy link
Member

frenzibyte commented Nov 27, 2024

Usually we would hold off such PRs until we have top-level skin settings in that case (similar to the yet-to-exist "hide perfect judgements" setting), but I'll be of the opinion on getting this one in given the argument of "I want my star fountains everywhere else except in gameplay" as mentioned in #30894 (comment) / #29792 (reply in thread), which cannot be resolved with any kind of user-side legacy skin hackery.

@peppy
Copy link
Member

peppy commented Nov 28, 2024

Just noting there was an alternative solution in #29792 (comment).

That's why this felt so familiar.

"Alternative" sure that's one way of looking at it 😅

Usually we would hold off such PRs until we have top-level skin settings

This is a user preference, not a skin setting. Skins should be providing every resource and it should be up to users to disable them.

@smoogipoo smoogipoo merged commit 5d7aafa into ppy:master Nov 28, 2024
9 of 10 checks passed
@HenintsoaSky HenintsoaSky deleted the star-fountains-toggle-setting branch November 28, 2024 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants