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.

Sign up for GitHub

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

Jump to bottom

Mod customization header/panel changes #29626

Merged
merged 10 commits into from
Sep 4, 2024
Merged

Mod customization header/panel changes #29626

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
be0e2ef
Removed on click event for expanding the Mod Customisation Header.
Fabep Aug 28, 2024
6adaf6a
Changed ModCustomisationPanelState names
Fabep Aug 28, 2024
363fc0d
Merge branch 'master' into master
Fabep Aug 30, 2024
1090107
Modify existing test coverage to demonstrate failure with touch
bdach Sep 2, 2024
5211e60
Fix mod customisation header being non-functional on mobile
bdach Sep 2, 2024
d99b231
Merge branch 'master' into Fabep/master
bdach Sep 2, 2024
16c2c14
Adjust tests further to match new UX
bdach Sep 2, 2024
e3457d8
Mod customisation header's color is now based on the state of the pan…
Fabep Sep 2, 2024
582ffcf
Woopsie! I accidentally added one too many semi-colons, so I moved it…
Fabep Sep 2, 2024
86309f4
Revert "Woopsie! I accidentally added one too many semi-colons, so I …
peppy Sep 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions osu.Game.Tests/Visual/UserInterface/TestSceneModCustomisationPanel.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,19 +58,19 @@ public void TestDisplay()
{
SelectedMods.Value = new[] { new OsuModDoubleTime() };
panel.Enabled.Value = true;
panel.ExpandedState.Value = ModCustomisationPanel.ModCustomisationPanelState.Expanded;
panel.ExpandedState.Value = ModCustomisationPanel.ModCustomisationPanelState.ExpandedByMod;
});
AddStep("set DA", () =>
{
SelectedMods.Value = new Mod[] { new OsuModDifficultyAdjust() };
panel.Enabled.Value = true;
panel.ExpandedState.Value = ModCustomisationPanel.ModCustomisationPanelState.Expanded;
panel.ExpandedState.Value = ModCustomisationPanel.ModCustomisationPanelState.ExpandedByMod;
});
AddStep("set FL+WU+DA+AD", () =>
{
SelectedMods.Value = new Mod[] { new OsuModFlashlight(), new ModWindUp(), new OsuModDifficultyAdjust(), new OsuModApproachDifferent() };
panel.Enabled.Value = true;
panel.ExpandedState.Value = ModCustomisationPanel.ModCustomisationPanelState.Expanded;
panel.ExpandedState.Value = ModCustomisationPanel.ModCustomisationPanelState.ExpandedByMod;
});
AddStep("set empty", () =>
{
Expand Down
16 changes: 1 addition & 15 deletions osu.Game/Overlays/Mods/ModCustomisationHeader.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,20 +112,6 @@ protected override void LoadComplete()
}, true);
}

protected override bool OnClick(ClickEvent e)
{
if (Enabled.Value)
{
ExpandedState.Value = ExpandedState.Value switch
{
ModCustomisationPanelState.Collapsed => ModCustomisationPanelState.Expanded,
_ => ModCustomisationPanelState.Collapsed
};
}

return base.OnClick(e);
}

private bool touchedThisFrame;

protected override bool OnTouchDown(TouchDownEvent e)
Expand All @@ -144,7 +130,7 @@ protected override bool OnHover(HoverEvent e)
if (Enabled.Value)
{
if (!touchedThisFrame && panel.ExpandedState.Value == ModCustomisationPanelState.Collapsed)
panel.ExpandedState.Value = ModCustomisationPanelState.ExpandedByHover;
panel.ExpandedState.Value = ModCustomisationPanelState.Expanded;
}

return base.OnHover(e);
Expand Down
6 changes: 3 additions & 3 deletions osu.Game/Overlays/Mods/ModCustomisationPanel.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,7 @@ protected override void Update()
{
base.Update();

if (ExpandedState.Value == ModCustomisationPanelState.ExpandedByHover
if (ExpandedState.Value == ModCustomisationPanelState.Expanded
&& !ReceivePositionalInputAt(inputManager.CurrentState.Mouse.Position)
&& inputManager.DraggedDrawable == null)
{
Expand All @@ -239,8 +239,8 @@ protected override void Update()
public enum ModCustomisationPanelState
{
Collapsed = 0,
ExpandedByHover = 1,
Expanded = 2,
Expanded = 1,
ExpandedByMod = 2,
}
}
}
2 changes: 1 addition & 1 deletion osu.Game/Overlays/Mods/ModSelectOverlay.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -368,7 +368,7 @@ private void updateCustomisation()
customisationPanel.Enabled.Value = true;

if (anyModPendingConfiguration)
customisationPanel.ExpandedState.Value = ModCustomisationPanel.ModCustomisationPanelState.Expanded;
customisationPanel.ExpandedState.Value = ModCustomisationPanel.ModCustomisationPanelState.ExpandedByMod;
}
else
{
Expand Down
Loading