Split inspector fixes:

* Fix inspector overpainting over reset-button
* Fix split inspector becoming active when changing modes but empty
* Fix top panel painting underneath split inspector resize bar

Author: alcomposer

Source/Sidebar/Inspector.h

@@ -155,6 +155,11 @@ class Inspector final : public Component {
         loadParameters(properties);
     }
 
+    bool isEmpty()
+    {
+        return properties.empty();
+    }
+
     bool loadParameters(SmallArray<ObjectParameters, 6>& objectParameters)
     {
         properties = objectParameters;

Source/Sidebar/Sidebar.cpp

@@ -108,21 +108,6 @@ void Sidebar::paint(Graphics& g)
     if (!sidebarHidden) {
         g.setColour(findColour(PlugDataColour::sidebarBackgroundColourId));
         g.fillRect(0, 30, getWidth(), getHeight());
-    }
-}
-
-void Sidebar::paintOverChildren(Graphics& g)
-{
-    g.setColour(findColour(PlugDataColour::toolbarOutlineColourId));
-    g.drawLine(0.5f, 30, 0.5f, getHeight() + 0.5f);
-
-    g.drawLine(dividerBounds.getX() + 4, dividerBounds.getCentreY(), dividerBounds.getRight() - 4, dividerBounds.getCentreY());
-
-    if (!sidebarHidden) {
-        g.drawLine(0, 30, getWidth(), 30);
-
-        g.setColour(findColour(PlugDataColour::toolbarOutlineColourId).withAlpha(0.5f));
-        g.drawLine(getWidth() - 30, 30, getWidth() - 30, getHeight() + 0.5f);
 
         auto panelName = panelNames[currentPanel];
         if (inspectorButton.isInspectorAuto() && inspector->isVisible())
@@ -131,7 +116,7 @@ void Sidebar::paintOverChildren(Graphics& g)
 
         if (inspectorButton.isInspectorPinned()) {
             auto inpectorPos = Point<int>(0, dividerFactor * getHeight());
-            if(!inspector->isVisible()) inpectorPos.setY(getHeight() - 30);
+            if (inspector->isEmpty()) inpectorPos.setY(getHeight() - 30);
             g.setColour(findColour(PlugDataColour::sidebarActiveBackgroundColourId));
             g.fillRect(inpectorPos.x, inpectorPos.y, getWidth() - 30, 30);
             auto inspectorTitle = inspector->getTitle();
@@ -142,6 +127,21 @@ void Sidebar::paintOverChildren(Graphics& g)
     }
 }
 
+void Sidebar::paintOverChildren(Graphics& g)
+{
+    g.setColour(findColour(PlugDataColour::toolbarOutlineColourId));
+    g.drawLine(0.5f, 30, 0.5f, getHeight() + 0.5f);
+
+    g.drawLine(dividerBounds.getX() + 4, dividerBounds.getCentreY(), dividerBounds.getRight() - 4, dividerBounds.getCentreY());
+
+    if (!sidebarHidden) {
+        g.drawLine(0, 30, getWidth(), 30);
+
+        g.setColour(findColour(PlugDataColour::toolbarOutlineColourId).withAlpha(0.5f));
+        g.drawLine(getWidth() - 30, 30, getWidth() - 30, getHeight() + 0.5f);
+    }
+}
+
 void Sidebar::settingsChanged(String const& name, var const& value)
 {
     if (name == "centre_sidepanel_buttons") {
@@ -192,15 +192,15 @@ void Sidebar::resized()
             }
             inspector->setBounds(bounds);
         } else {
-            auto bottomB = bounds.removeFromBottom(dividerPos);
+            auto bottomB = bounds.removeFromBottom(inspector->isEmpty() ? 30 : dividerPos);
             auto resetB = bottomB.removeFromTop(30);
             inspector->setBounds(bottomB);
             auto const resetBounds = resetB.removeFromLeft(30);
             if (extraSettingsButton)
                 extraSettingsButton->setVisible(true);
             if (resetInspectorButton) {
                 resetInspectorButton->setBounds(resetBounds);
-                resetInspectorButton->setVisible(true);
+                resetInspectorButton->setVisible(!inspector->isEmpty());
             }
         }
     } else {
@@ -331,7 +331,7 @@ void Sidebar::showPanel(SidePanel const panelToShow)
         break;
     case SidePanel::InspectorPan:
         if (!sidebarHidden) {
-            auto const isVisible = (inspectorButton.isInspectorPinned() || inspectorButton.isInspectorAuto()) && lastParameters.not_empty();
+            auto const isVisible = (inspectorButton.isInspectorPinned() || (inspectorButton.isInspectorAuto() && !inspector->isEmpty()));
             if (!areParamObjectsAllValid()) {
                 clearInspector();
             }
@@ -455,7 +455,7 @@ void Sidebar::showParameters(SmallArray<Component*>& objects, SmallArray<ObjectP
 
     auto const haveParams = showOnSelect && params.not_empty() && activeParams;
 
-    auto const isVis = (inspectorButton.isInspectorAuto() || inspectorButton.isInspectorPinned()) && params.not_empty() && showOnSelect && activeParams;
+    auto const isVis = (inspectorButton.isInspectorAuto() && params.not_empty() && showOnSelect && activeParams) || inspectorButton.isInspectorPinned();
 
     // Reset console notifications if the inspector is not visible and console is
     if (!isVis && currentPanel == SidePanel::ConsolePan) {
@@ -526,7 +526,7 @@ void Sidebar::updateExtraSettingsButton()
 
 void Sidebar::hideParameters()
 {
-    if (inspectorButton.isInspectorAuto() || inspectorButton.isInspectorPinned()) {
+    if (inspectorButton.isInspectorAuto()) {
         inspector->setVisible(false);
     }