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

Decouple FilterControl.Ruleset from the game-wide ruleset bindable #2471

Conversation

@UselessToucan
Copy link
Contributor

@UselessToucan UselessToucan commented Apr 29, 2018

@peppy peppy added the resolves issue label May 2, 2018
@smoogipoo
Copy link
Contributor

@smoogipoo smoogipoo commented May 2, 2018

I don't think direct and osu! should share the game-wide ruleset bindable in the first place. Agree, @peppy ?

@peppy
Copy link
Member

@peppy peppy commented May 2, 2018

I think that's the direction we were going in, yeah.

Also I thought this functionality already existed at an OsuScreen level or similar? I know that these buttons were already disabled when in play mode.

@smoogipoo
Copy link
Contributor

@smoogipoo smoogipoo commented May 2, 2018

The toolbar buttons are, but the osu!direct buttons weren't disabled, and they share the game-wide bindable hence triggering the exception. But I can't imagine why you'd want to disable the direct ones.

@smoogipoo
Copy link
Contributor

@smoogipoo smoogipoo commented May 2, 2018

Instead of this, I propose we decouple osu!direct from the game-wide ruleset bindable. It should do something like: Ruleset.Value = game?.Ruleset.Value ?? rulesets.GetRuleset(0) rather than a .BindTo().

osu!direct should dependency cache a ruleset bindable for children to use, if anything else other than this filter control needs it.

@smoogipoo smoogipoo added this to the May 2018 milestone May 2, 2018
Copy link
Contributor

@smoogipoo smoogipoo left a comment

as commented

@UselessToucan
Copy link
Contributor Author

@UselessToucan UselessToucan commented May 3, 2018

…reen
@smoogipoo smoogipoo removed this from the May 2018 milestone May 7, 2018
@UselessToucan UselessToucan changed the title Disable RulesetToggleButton whenever the ruleset can't be changed Decouple FilterControl.Ruleset from the game-wide ruleset bindable May 8, 2018
Copy link
Contributor

@smoogipoo smoogipoo left a comment

lgtm!

…reen
@smoogipoo smoogipoo merged commit 3c68935 into ppy:master May 9, 2018
1 check passed
1 check passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@UselessToucan UselessToucan deleted the UselessToucan:change_ruleset_via_direct_while_on_scorescreen branch May 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.