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

Rename "Aim Assist" to "Magnetised" to better suit the mod's behaviour #17588

Merged
merged 2 commits into from
Apr 1, 2022

Conversation

peppy
Copy link
Member

@peppy peppy commented Apr 1, 2022

As proposed in #17375.

osu! 2022-04-01 at 03 17 12

smoogipoo
smoogipoo previously approved these changes Apr 1, 2022
public override double ScoreMultiplier => 1;
public override Type[] IncompatibleMods => new[] { typeof(OsuModAutopilot), typeof(OsuModWiggle), typeof(OsuModTransform), typeof(ModAutoplay), typeof(OsuModRelax) };

private IFrameStableClock gameplayClock;

[SettingSource("Assist strength", "How much this mod will assist you.", 0)]
[SettingSource("Attraction strength", "How strong the pull is.", 0)]
public BindableFloat AssistStrength { get; } = new BindableFloat(0.5f)
Copy link
Member Author

Choose a reason for hiding this comment

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

I didn't rename the property here to avoid breaking compatibility, but then realised the mod name is changing so that's a non issue. Think we should rename this to AttractionStrength while here, or not important?

Copy link
Contributor

Choose a reason for hiding this comment

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

Sounds good

@smoogipoo smoogipoo merged commit 0a86bf6 into ppy:master Apr 1, 2022
@peppy peppy deleted the aim-assist-rename branch April 1, 2022 07:19
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.

2 participants