Adress the first round of Angelaccio's review comments

- Split the viewContainers(bool includeInActive) into two methods
    without parameters
- Prevent users from accidently hiding all Url Navigators by
    preventing the dangerous action and then displaying a helpful
    message instead
Unrelated to review comments: Remove a useless line of code

1 files changed, 6 insertions, 2 deletions

diff --git a/src/dolphinviewcontainer.cpp b/src/dolphinviewcontainer.cpp
index da3504187..3cdad13fb 100644
--- a/src/dolphinviewcontainer.cpp
+++ b/src/dolphinviewcontainer.cpp
@@ -367,10 +367,13 @@ void DolphinViewContainer::disconnectUrlNavigator()
     updateNavigatorWidgetVisibility();
 }
 
-void DolphinViewContainer::showMessage(const QString& msg, MessageType type)
+KMessageWidget *DolphinViewContainer::showMessage(const QString& msg, MessageType type)
 {
     if (msg.isEmpty()) {
-        return;
+        return m_messageWidget;
+    }
+    for (auto action : m_messageWidget->actions()) {
+        m_messageWidget->removeAction(action);
     }
 
     m_messageWidget->setText(msg);
@@ -396,6 +399,7 @@ void DolphinViewContainer::showMessage(const QString& msg, MessageType type)
         m_messageWidget->hide();
     }
     m_messageWidget->animatedShow();
+    return m_messageWidget;
 }
 
 void DolphinViewContainer::readSettings()