Skip to content

Commit

Permalink
oop
Browse files Browse the repository at this point in the history
  • Loading branch information
Flutterish committed Sep 3, 2022
1 parent dc582c9 commit 44efff8
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions osu.Game.Rulesets.RurusettoAddon/RurusettoIcon.cs
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ void errDefer ( Action action ) {
}

overlayContent.Add( overlay );
errDefer( () => overlayContent.Remove( overlay ) );
errDefer( () => overlayContent.Remove( overlay, false ) );

// https://github.com/ppy/osu/blob/edf5e558aca6cd75e70b510a5f0dd233d6cfcb90/osu.Game/Overlays/Toolbar/Toolbar.cs#L89
// leveraging an "easy" hack to get the container with toolbar buttons
Expand All @@ -95,7 +95,7 @@ void errDefer ( Action action ) {

var button = new RurusettoToolbarButton();
buttonsContainer.Insert( -1, button );
errDefer( () => buttonsContainer.Remove( button ) );
errDefer( () => buttonsContainer.Remove( button, false ) );

// https://github.com/ppy/osu/blob/edf5e558aca6cd75e70b510a5f0dd233d6cfcb90/osu.Game/OsuGame.cs#L855
// add overlay hiding, since osu does it manually
Expand Down
4 changes: 2 additions & 2 deletions osu.Game.Rulesets.RurusettoAddon/UI/Listing/ListingTab.cs
Original file line number Diff line number Diff line change
Expand Up @@ -98,9 +98,9 @@ public override void Add ( DrawableListingEntry drawable ) {
rulesets.Add( drawable, drawable.Ruleset );
}

public override bool Remove ( DrawableListingEntry drawable ) {
public override bool Remove ( DrawableListingEntry drawable, bool disposeImmediately ) {
rulesets.Remove( drawable );
return base.Remove( drawable );
return base.Remove( drawable, disposeImmediately );
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ protected override void PopOutComplete () {

if ( Header.Current.Value.Tab is null && Header.CurrentCategory.Value == Header.ListingTab.Category ) {
foreach ( var i in infoTabs ) {
tabContainer.Remove( i.Value );
tabContainer.Remove( i.Value, false );
i.Value.Dispose();
}
userTabs.Clear();
Expand Down
2 changes: 1 addition & 1 deletion osu.Game.Rulesets.RurusettoAddon/UI/RulesetLogo.cs
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ private void load ( OverlayColourProvider colours ) {
AddInternal( logo );
}
catch {
RemoveInternal( logo );
RemoveInternal( logo, false );
ruleset.RequestDarkLogo( AddInternal, AddInternal, useLocalIcon: false );
}
}, fallback => AddInternal( fallback ) );
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,6 @@
<None Remove="Resources\Textures\oh_no.png" />
</ItemGroup>
<ItemGroup>
<PackageReference Include="ppy.osu.Game" Version="2022.902.2" />
<PackageReference Include="ppy.osu.Game" Version="2022.902.1" />
</ItemGroup>
</Project>

0 comments on commit 44efff8

Please sign in to comment.