From 71e2815e7ec670693bf94ad8885e10b13015f338 Mon Sep 17 00:00:00 2001
From: Salman Ahmed <frenzibyte@gmail.com>
Date: Tue, 22 Jun 2021 12:05:17 +0300
Subject: [PATCH] Update and improve code documentation

Co-authored-by: Dean Herbert <pe@ppy.sh>
---
 osu.Game/Skinning/RulesetSkinProvidingContainer.cs | 5 ++---
 osu.Game/Skinning/SkinManager.cs                   | 4 ++--
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/osu.Game/Skinning/RulesetSkinProvidingContainer.cs b/osu.Game/Skinning/RulesetSkinProvidingContainer.cs
index f1cc3df3de..c48aeca99a 100644
--- a/osu.Game/Skinning/RulesetSkinProvidingContainer.cs
+++ b/osu.Game/Skinning/RulesetSkinProvidingContainer.cs
@@ -13,8 +13,7 @@ namespace osu.Game.Skinning
 {
     /// <summary>
     /// A type of <see cref="SkinProvidingContainer"/> specialized for <see cref="DrawableRuleset"/> and other gameplay-related components.
-    /// Providing access to the <see cref="SkinManager"/> skin sources and the beatmap skin each surrounded with the ruleset legacy skin transformer.
-    /// While also limiting lookups from falling back to any parent <see cref="ISkinSource"/>s out of this container.
+    /// Providing access to parent skin sources and the beatmap skin each surrounded with the ruleset legacy skin transformer.
     /// </summary>
     public class RulesetSkinProvidingContainer : SkinProvidingContainer
     {
@@ -22,7 +21,7 @@ namespace osu.Game.Skinning
         protected readonly IBeatmap Beatmap;
 
         /// <remarks>
-        /// This container already re-exposes all <see cref="SkinManager"/> skin sources in a ruleset-usable form.
+        /// This container already re-exposes all parent <see cref="ISkinSource"/> sources in a ruleset-usable form.
         /// Therefore disallow falling back to any parent <see cref="ISkinSource"/> any further.
         /// </remarks>
         protected override bool AllowFallingBackToParent => false;
diff --git a/osu.Game/Skinning/SkinManager.cs b/osu.Game/Skinning/SkinManager.cs
index 3234cca0ac..4cde4cd2b8 100644
--- a/osu.Game/Skinning/SkinManager.cs
+++ b/osu.Game/Skinning/SkinManager.cs
@@ -34,8 +34,8 @@ namespace osu.Game.Skinning
     /// Handles the storage and retrieval of <see cref="Skin"/>s.
     /// </summary>
     /// <remarks>
-    /// This is also exposed and cached as <see cref="ISkinSource"/> on a game-wide level for general components across the game.
-    /// Lookups from gameplay components are instead covered by <see cref="RulesetSkinProvidingContainer"/>, and are never hit here.
+    /// This is also exposed and cached as <see cref="ISkinSource"/> to allow for any component to potentially have skinning support.
+    /// For gameplay components, see <see cref="RulesetSkinProvidingContainer"/> which adds extra legacy and toggle logic that may affect the lookup process.
     /// </remarks>
     [ExcludeFromDynamicCompile]
     public class SkinManager : ArchiveModelManager<SkinInfo, SkinFileInfo>, ISkinSource, IStorageResourceProvider