Revert "* Temporarily disables #331 (#2587)"

This reverts commit 6b898fe.

Author: giduac

.github/workflows/release.yml

@@ -863,3 +863,4 @@ jobs:
           Invoke-RestMethod -Uri $env:DISCORD_WEBHOOK_NIGHTLY -Method Post -Body $payload -ContentType "application/json"
         env:
           DISCORD_WEBHOOK_NIGHTLY: ${{ secrets.DISCORD_WEBHOOK_NIGHTLY }}
+

Documents/Changelog/Changelog.md

@@ -8,6 +8,7 @@
 * Resolved [#2542](https://github.com/Krypton-Suite/Standard-Toolkit/issues/2542), `KryptonForm` cannot be resized by dragging upper corners.
 * Implemented [#2575](https://github.com/Krypton-Suite/Standard-Toolkit/issues/2575), **[Breaking Change]** `CommonHelper.DesignMode()` behaviour changed, see issue for details.
 * Implemented [#2559](https://github.com/Krypton-Suite/Standard-Toolkit/issues/2559), Allow `release.yml` to automatically push packages
+* Implemented [#331](https://github.com/Krypton-Suite/Standard-Toolkit/issues/331), Make the "No Tab in a ribbon Solution" An Actual Designer Tool
 * Resolved [#2331](https://github.com/Krypton-Suite/Standard-Toolkit/issues/2331), Correction of edges, 3D effect and color adjustment in `KryptonRibbon` in Office 2010 themes.
 * Resolved [#2548](https://github.com/Krypton-Suite/Standard-Toolkit/issues/2548), Fixed color assignment for `GroupSeparatorLight`, `QATButtonDarkColor` and `QATButtonLightColor` in `PopulateFromBase`.
 * Resolved [#2461](https://github.com/Krypton-Suite/Standard-Toolkit/issues/2461), (feat) Add `KryptonCalcInput` edit+dropdown control.

Source/Krypton Components/Krypton.Ribbon/Controls Ribbon/KryptonRibbon.cs

@@ -83,10 +83,7 @@ public RibbonButtonSpecAnyCollection(KryptonRibbon owner)
     // Properties
     private bool _minimizedMode;
     private bool _showMinimizeButton;
-
-    // ToDo: Reinvestigate for #331 in the future, see https://github.com/Krypton-Suite/Standard-Toolkit/issues/331 & https://github.com/Krypton-Suite/Standard-Toolkit/issues/2584 for more information
-    //private bool _showTabs;
-
+    private bool _showTabs;
     private string _selectedContext;
     private Size _hideRibbonSize;
     private QATLocation _qatLocation;
@@ -904,8 +901,7 @@ public QATLocation QATLocation
     /// </summary>
     public void ResetQATUserChange() => QATUserChange = true;
 
-    // ToDo: Reinvestigate for #331 in the future, see https://github.com/Krypton-Suite/Standard-Toolkit/issues/331 & https://github.com/Krypton-Suite/Standard-Toolkit/issues/2584 for more information
-    /*/// <summary>
+    /// <summary>
     /// Gets and sets a value indicating if ribbon tabs are visible.
     /// </summary>
     [Category(@"Values")]
@@ -929,7 +925,7 @@ public bool ShowTabs
     /// <summary>
     /// Resets the ShowTabs property to its default value.
     /// </summary>
-    public void ResetShowTabs() => ShowTabs = true;*/
+    public void ResetShowTabs() => ShowTabs = true;
 
     /// <summary>
     /// Gets and sets a value indicating if a minimize/expand button appears on the ribbon tab ara.
@@ -1555,7 +1551,7 @@ protected override void OnGotFocus(EventArgs e)
         {
             newFocus = TabsArea?.LayoutTabs.GetViewForRibbonTab(SelectedTab);
         }
-        else if (!RealMinimizedMode /*&& ShowTabs*/)
+        else if (!RealMinimizedMode && ShowTabs)
         {
             // Only try to focus first tab if tabs are visible
             newFocus = TabsArea?.LayoutTabs.GetViewForFirstRibbonTab();
@@ -1831,8 +1827,7 @@ internal void UpdateBackStyle()
         }
     }
 
-    // ToDo: Reinvestigate for #331 in the future, see https://github.com/Krypton-Suite/Standard-Toolkit/issues/331 & https://github.com/Krypton-Suite/Standard-Toolkit/issues/2584 for more information
-    /*/// <summary>
+    /// <summary>
     /// Updates the visibility of the tabs area and caption area based on the ShowTabs property.
     /// </summary>
     private void UpdateTabVisibility()
@@ -1870,7 +1865,7 @@ private void UpdateTabVisibility()
                 _ribbonDocker.Remove(CaptionArea);
             }
         }
-    }*/
+    }
 
     [DesignerSerializationVisibility(DesignerSerializationVisibility.Hidden)]
     internal ButtonStyle ScrollerStyle
@@ -2606,7 +2601,7 @@ private void AssignDefaultFields()
         MinimizedMode = false;
         ScrollerStyle = ButtonStyle.Standalone;
         ShowMinimizeButton = true;
-        //ShowTabs = true;
+        ShowTabs = true;
         QATLocation = QATLocation.Above;
         QATUserChange = true;
         LostFocusLosesKeyboard = true;
@@ -2731,14 +2726,13 @@ private void CreateViewManager()
         _ribbonDocker.Add(GroupsArea, ViewDockStyle.Fill);
         _ribbonDocker.Add(_minimizeBar, ViewDockStyle.Bottom);
         _ribbonDocker.Add(_qatBelowRibbon, ViewDockStyle.Bottom);
-
-        // ToDo: Reinvestigate for #331 in the future, see https://github.com/Krypton-Suite/Standard-Toolkit/issues/331 & https://github.com/Krypton-Suite/Standard-Toolkit/issues/2584 for more information
+        
         // Add tabs area and caption area only if tabs are visible
-        /*if (_showTabs)
+        if (_showTabs)
         {
             _ribbonDocker.Add(TabsArea, ViewDockStyle.Top);
             _ribbonDocker.Add(CaptionArea, ViewDockStyle.Top);
-        }*/
+        }
 
         // The root contains the top and fills out with the panel areas
         _rootDocker.Add(MainPanel, ViewDockStyle.Fill);
@@ -2839,7 +2833,7 @@ private void ValidateSelectedTab()
         KryptonRibbonTab? newSelection = null;
 
         // If tabs are hidden (toolbar mode), don't auto-select any tab
-        /*if (!ShowTabs)
+        if (!ShowTabs)
         {
             // In toolbar mode, we allow SelectedTab to remain null
             // Only validate if there's a selected tab that's no longer valid
@@ -2853,7 +2847,7 @@ private void ValidateSelectedTab()
                 newSelection = SelectedTab; // Keep current selection (including null)
             }
         }
-        else*/ if (!RealMinimizedMode)
+        else if (!RealMinimizedMode)
         {
             if ((SelectedTab == null) ||                            // If there is no selection...
                 (!SelectedTab.Visible && !InDesignHelperMode) ||    // Or the selection is no longer visible

Source/Krypton Components/Krypton.Ribbon/Designers/Designers/KryptonRibbonDesigner.cs

@@ -24,12 +24,8 @@ internal class KryptonRibbonDesigner : ParentControlDesigner
     private DesignerVerb _clearTabsVerb;
     private DesignerVerb _noTabVerb;
     private DesignerVerb _setTabVerb;
-
-    // ToDo: Reinvestigate for #331 in the future, see https://github.com/Krypton-Suite/Standard-Toolkit/issues/331 & https://github.com/Krypton-Suite/Standard-Toolkit/issues/2584 for more information
-    // Disabled ShowTabs feature temporarily due to rendering issues with KryptonRibbonContext
-    // private DesignerVerb _hideTabsVerb;
-    // private DesignerVerb _showTabsVerb;
-
+    private DesignerVerb _hideTabsVerb;
+    private DesignerVerb _showTabsVerb;
     private bool _lastHitTest;
 
     #endregion
@@ -151,12 +147,9 @@ public override DesignerVerbCollection Verbs
                 _clearTabsVerb = new DesignerVerb(@"Clear Tabs", OnClearTabs);
                 _noTabVerb = new DesignerVerb(@"Clear Tab Selection", OnNoTab);
                 _setTabVerb = new DesignerVerb(@"Select First Tab", OnSetTab);
-
-                // ToDo: Reinvestigate for #331 in the future, see https://github.com/Krypton-Suite/Standard-Toolkit/issues/331 & https://github.com/Krypton-Suite/Standard-Toolkit/issues/2584 for more information
-                // Disabled ShowTabs feature temporarily due to rendering issues with KryptonRibbonContext
-                // _hideTabsVerb = new DesignerVerb(@"Hide Tab Headers", OnHideTabs);
-                // _showTabsVerb = new DesignerVerb(@"Show Tab Headers", OnShowTabs);
-                _verbs.AddRange(new[] { _toggleHelpersVerb, _addTabVerb, _clearTabsVerb, _noTabVerb, _setTabVerb /*, _hideTabsVerb, _showTabsVerb*/ });
+                _hideTabsVerb = new DesignerVerb(@"Hide Tab Headers", OnHideTabs);
+                _showTabsVerb = new DesignerVerb(@"Show Tab Headers", OnShowTabs);
+                _verbs.AddRange(new[] { _toggleHelpersVerb, _addTabVerb, _clearTabsVerb, _noTabVerb, _setTabVerb, _hideTabsVerb, _showTabsVerb });
             }
 
             UpdateVerbStatus();
@@ -285,11 +278,9 @@ private void UpdateVerbStatus()
                 _noTabVerb.Enabled = true;
                 _setTabVerb.Enabled = false;
             }
-
-            // ToDo: Reinvestigate for #331 in the future, see https://github.com/Krypton-Suite/Standard-Toolkit/issues/331 & https://github.com/Krypton-Suite/Standard-Toolkit/issues/2584 for more information
-            // Disabled ShowTabs feature temporarily due to rendering issues with KryptonRibbonContext
+            
             // Update the Hide Tabs / Show Tabs verbs based on current state
-            /*if (_ribbon.ShowTabs)
+            if (_ribbon.ShowTabs)
             {
                 _hideTabsVerb.Enabled = true;
                 _showTabsVerb.Enabled = false;
@@ -298,7 +289,7 @@ private void UpdateVerbStatus()
             {
                 _hideTabsVerb.Enabled = false;
                 _showTabsVerb.Enabled = true;
-            }*/
+            }
         }
     }
 
@@ -464,15 +455,13 @@ private void OnSetTab(object? sender, EventArgs e)
         }
     }
 
-    // ToDo: Reinvestigate for #331 in the future, see https://github.com/Krypton-Suite/Standard-Toolkit/issues/331 & https://github.com/Krypton-Suite/Standard-Toolkit/issues/2584 for more information
-    /*//Disabled ShowTabs feature temporarily due to rendering issues with KryptonRibbonContext
-     /// <summary>
-     /// Handles the "Hide Tabs" designer verb click.
-     /// Hides the ribbon tabs to create a toolbar-like interface.
-     /// </summary>
-     /// <param name="sender">The source of the event.</param>
-     /// <param name="e">An EventArgs that contains the event data.</param>
-     private void OnHideTabs(object? sender, EventArgs e)
+    /// <summary>
+    /// Handles the "Hide Tabs" designer verb click.
+    /// Hides the ribbon tabs to create a toolbar-like interface.
+    /// </summary>
+    /// <param name="sender">The source of the event.</param>
+    /// <param name="e">An EventArgs that contains the event data.</param>
+    private void OnHideTabs(object? sender, EventArgs e)
     {
         if (_ribbon is null)
         {
@@ -503,14 +492,13 @@ private void OnHideTabs(object? sender, EventArgs e)
         }
     }
 
-     // Disabled ShowTabs feature temporarily due to rendering issues with KryptonRibbonContext
-     /// <summary>
-     /// Handles the "Show Tabs" designer verb click.
-     /// Shows the ribbon tabs to restore normal ribbon interface.
-     /// </summary>
-     /// <param name="sender">The source of the event.</param>
-     /// <param name="e">An EventArgs that contains the event data.</param>
-     private void OnShowTabs(object? sender, EventArgs e)
+    /// <summary>
+    /// Handles the "Show Tabs" designer verb click.
+    /// Shows the ribbon tabs to restore normal ribbon interface.
+    /// </summary>
+    /// <param name="sender">The source of the event.</param>
+    /// <param name="e">An EventArgs that contains the event data.</param>
+    private void OnShowTabs(object? sender, EventArgs e)
     {
         if (_ribbon is null)
         {
@@ -539,7 +527,7 @@ private void OnShowTabs(object? sender, EventArgs e)
 
             UpdateVerbStatus();
         }
-    }*/
+    }
 
     private void OnRibbonMouseUp(object? sender, MouseEventArgs e)
     {