Back from a mod card returns to the mods grid; collections link in its own row, status and version no longer cut off; mod cover

src/ModHubScreen.cs

@@ -25,8 +25,9 @@ namespace Mods.ModManager
     /// The hub also carries the mockup's cross links: a collection link on a
     /// mod card switches to the collections tab, a "not found" problem in the
     /// editor jumps back to the mods tab with the id filled into the search
-    /// box (<see cref="ModManagerScreen.SetSearch"/>), and Escape walks back
-    /// editor → gallery → mods tab.
+    /// box (<see cref="ModManagerScreen.SetSearch"/>). The back button and Escape
+    /// walk back one level at a time — editor → gallery, an open mod card → the
+    /// mods grid — and only then leave to the menu.
     /// </summary>
     public partial class ModHubScreen : ModWidget
     {
@@ -223,7 +224,7 @@ namespace Mods.ModManager
                 return;
             }
 
-            _back.Text = TranslationServer.Translate("mod_manager/hub_back_to_menu");
+            _back.Text = TranslationServer.Translate(ModOpen ? "mod_manager/hub_back_to_mods" : "mod_manager/hub_back_to_menu");
             _back.TooltipText = _back.Text;
             _title.Text = TranslationServer.Translate("mod_manager/menu_title");
             _segment.Visible = true;
@@ -235,10 +236,13 @@ namespace Mods.ModManager
             _galleryRefresh.Visible = collections;
         }
 
+        /// <summary>Back walks one level up: scheme editor → games gallery, mod card → mods, then the menu.</summary>
         private void OnBackPressed()
         {
             if (_editorOpen)
                 ShowGallery();
+            else if (ModOpen)
+                _mods.CloseDetail();
             else
                 QueueFree();
         }
@@ -262,8 +266,11 @@ namespace Mods.ModManager
             _mods.SetAnchorsAndOffsetsPreset(LayoutPreset.FullRect);
             _mods.EnsureConstructed();
             _mods.CollectionsLinkRequested += () => ShowTab(TabCollections);
+            _mods.DetailToggled += UpdateHeader;
         }
 
+        private bool ModOpen => _activeTab == TabMods && _mods != null && _mods.IsDetailOpen;
+
         private void EnsureGallery()
         {
             if (_gallery != null)
@@ -350,7 +357,7 @@ namespace Mods.ModManager
 
         /* ═══════════════ Escape chain ═══════════════ */
 
-        /// <summary>Escape walks back one level: editor → gallery → mods tab.</summary>
+        /// <summary>Escape walks back one level: editor → gallery → mods tab, mod card → mods grid.</summary>
         public override void _UnhandledInput(InputEvent @event)
         {
             if (!@event.IsActionPressed("ui_cancel"))
@@ -361,6 +368,11 @@ namespace Mods.ModManager
                 ShowGallery();
                 GetViewport().SetInputAsHandled();
             }
+            else if (ModOpen)
+            {
+                _mods.CloseDetail();
+                GetViewport().SetInputAsHandled();
+            }
             else if (_activeTab == TabCollections)
             {
                 ShowTab(TabMods);

src/ModManagerScreen.cs

@@ -52,6 +52,12 @@ namespace Mods.ModManager
         /// </summary>
         public bool Embedded { get; set; }
 
+        /// <summary>Whether a mod card is open over the grid.</summary>
+        public bool IsDetailOpen => _detail != null && _detail.Visible;
+
+        /// <summary>Raised when a mod card opens or closes (the hub relabels its back button).</summary>
+        public event Action DetailToggled;
+
         /// <summary>Raised when a tile or the detail page asks to jump to the collections tab.</summary>
         public event Action CollectionsLinkRequested;
 
@@ -153,7 +159,6 @@ namespace Mods.ModManager
             _grid.EnabledToggled += OnEnabledToggled;
             _grid.VersionChanged += OnVersionChanged;
             _grid.CollectionsLinkRequested += OnCollectionsLinkRequested;
-            _detail.BackRequested += OnDetailBackRequested;
             _detail.ActionPressed += OnActionPressed;
             _detail.DeleteRequested += OnDeleteRequested;
             _detail.EnabledToggled += OnEnabledToggled;
@@ -172,7 +177,6 @@ namespace Mods.ModManager
             _grid.EnabledToggled -= OnEnabledToggled;
             _grid.VersionChanged -= OnVersionChanged;
             _grid.CollectionsLinkRequested -= OnCollectionsLinkRequested;
-            _detail.BackRequested -= OnDetailBackRequested;
             _detail.ActionPressed -= OnActionPressed;
             _detail.DeleteRequested -= OnDeleteRequested;
             _detail.EnabledToggled -= OnEnabledToggled;
@@ -315,10 +319,17 @@ namespace Mods.ModManager
             QueueFree();
         }
 
-        private void OnDetailBackRequested()
+        /// <summary>
+        /// Closes an open mod card and shows the grid again. The hub's back button and
+        /// Escape call it first, so "back" from a mod returns to the mods, not to the menu.
+        /// </summary>
+        public void CloseDetail()
         {
+            if (!_detail.Visible)
+                return;
             _detail.Visible = false;
             _grid.Visible = true;
+            DetailToggled?.Invoke();
         }
 
         private void OnCardClicked(ModCardData data)
@@ -326,6 +337,7 @@ namespace Mods.ModManager
             _detail.Open(data);
             _detail.Visible = true;
             _grid.Visible = false;
+            DetailToggled?.Invoke();
         }
 
         private void OnSearchChanged(string text)
@@ -388,7 +400,7 @@ namespace Mods.ModManager
                 if (updated != null)
                     _detail.Open(updated);
                 else
-                    OnDetailBackRequested();
+                    CloseDetail();
             }
         }
 
@@ -486,12 +498,7 @@ namespace Mods.ModManager
             _restartRequired = true;
             _statusMessage = null;
 
-            if (_detail.Visible)
-            {
-                _detail.Visible = false;
-                _grid.Visible = true;
-            }
-
+            CloseDetail();
             RefreshAll();
         }
 

src/UI/ModCard.cs

@@ -29,7 +29,7 @@ namespace Mods.ModManager
         private OptionButton _versions;
         private Button _action;
         private CheckButton _enabled;
-        private HBoxContainer _metaRow;
+        private HBoxContainer _linkRow;
         private Button _collectionsLink;
         private bool _updatingEnabled;
         private bool _initialized;
@@ -68,7 +68,7 @@ namespace Mods.ModManager
             _versions = GetNode<OptionButton>("VBox/Body/Box/MetaRow/VersionHost/Versions");
             _action = GetNode<Button>("VBox/Body/Box/ActionRow/ActionHost/Action");
             _enabled = GetNode<CheckButton>("VBox/Body/Box/ActionRow/EnabledHost/Enabled");
-            _metaRow = GetNode<HBoxContainer>("VBox/Body/Box/MetaRow");
+            _linkRow = GetNode<HBoxContainer>("VBox/Body/Box/LinkRow");
 
             _author.AddThemeColorOverride("font_color", UiTokens.Muted);
             _source.AddThemeColorOverride("font_color", UiTokens.Muted);
@@ -106,7 +106,8 @@ namespace Mods.ModManager
             {
                 _status.Text = ModStatusNames.Translate(_data.Status);
                 _status.AddThemeColorOverride("font_color", ModStatusNames.Color(_data.Status));
-                _status.TooltipText = string.Empty;
+                // A long status next to the version combo may end in an ellipsis.
+                _status.TooltipText = _status.Text;
             }
 
             UpdateActionButton();
@@ -160,13 +161,18 @@ namespace Mods.ModManager
             }
 
             _versions.Selected = selectedIndex;
+            // "v1.2.0 · latest · installed" does not fit the narrow combo: it ends in an
+            // ellipsis (clip_text in the scene), the full label is in the tooltip.
+            _versions.TooltipText = _versions.GetItemText(selectedIndex);
         }
 
         /// <summary>
-        /// Builds the flat "in N collections" cross-link shown between the
-        /// status label and the version combo. The link is part of the hub
-        /// mockup: pressing it jumps to the collections tab. Hidden entirely
-        /// for mods that no playable game references.
+        /// Builds the flat "in N collections" cross-link in its own row above the
+        /// status. The link is part of the hub mockup: pressing it jumps to the
+        /// collections tab. It used to share the status row, where its text took the
+        /// width the status needed ("Installed" was cut off); the row is reserved on
+        /// every card, empty for mods no playable game references, so all tiles keep
+        /// one size.
         /// </summary>
         private void BuildCollectionsLink()
         {
@@ -183,8 +189,7 @@ namespace Mods.ModManager
             foreach (var state in new[] { "normal", "hover", "pressed", "disabled", "focus" })
                 _collectionsLink.AddThemeStyleboxOverride(state, new StyleBoxEmpty());
             _collectionsLink.Pressed += () => CollectionsLinkRequested?.Invoke();
-            _metaRow.AddChild(_collectionsLink);
-            _metaRow.MoveChild(_collectionsLink, 1);
+            _linkRow.AddChild(_collectionsLink);
         }
 
         private void UpdateCollectionsLink()
@@ -282,6 +287,7 @@ namespace Mods.ModManager
                 return;
 
             _data.SelectedDownload = _data.Downloads[(int)index];
+            _versions.TooltipText = _versions.GetItemText((int)index);
             _data.RecomputeStatus();
             RefreshState();
             VersionChanged?.Invoke(_data);

src/UI/ModDetailView.cs

@@ -21,7 +21,6 @@ namespace Mods.ModManager
         private bool _nodesReady;
 
         private TextureRect _coverImage;
-        private Button _back;
         private Label _title;
         private Label _author;
         private Label _meta;
@@ -40,9 +39,6 @@ namespace Mods.ModManager
         private VBoxContainer _conflictsBox;
         private Label _conflictsText;
 
-        /// <summary>Raised by the back button.</summary>
-        public event Action BackRequested;
-
         /// <summary>Raised when the primary contextual action is pressed.</summary>
         public event Action<ModCardData> ActionPressed;
 
@@ -72,7 +68,6 @@ namespace Mods.ModManager
             _nodesReady = true;
 
             _coverImage = GetNode<TextureRect>("Scroll/Content/Hero/HeroMargin/HeroRow/Cover/Image");
-            _back = GetNode<Button>("Scroll/Content/Hero/HeroMargin/HeroRow/Info/Back");
             _title = GetNode<Label>("Scroll/Content/Hero/HeroMargin/HeroRow/Info/Title");
             _author = GetNode<Label>("Scroll/Content/Hero/HeroMargin/HeroRow/Info/Author");
             _meta = GetNode<Label>("Scroll/Content/Hero/HeroMargin/HeroRow/Info/Meta");
@@ -91,8 +86,6 @@ namespace Mods.ModManager
             _conflictsBox = GetNode<VBoxContainer>("Scroll/Content/ConflictsBox");
             _conflictsText = GetNode<Label>("Scroll/Content/ConflictsBox/Text");
 
-            _back.Text = TranslationServer.Translate("mod_manager/back_to_list");
-            _back.Pressed += OnBackPressed;
             _versions.ItemSelected += OnVersionSelected;
 
             _author.AddThemeColorOverride("font_color", UiTokens.Muted);
@@ -298,8 +291,6 @@ namespace Mods.ModManager
                 _coverImage.Texture = texture;
         }
 
-        private void OnBackPressed() => BackRequested?.Invoke();
-
         private void OnPrimaryPressed() => ActionPressed?.Invoke(_data);
 
         private void OnDeletePressed() => DeleteRequested?.Invoke(_data);