From b3b1ac69468c60c2a26468b0fa4a9d8e15e237bb Mon Sep 17 00:00:00 2001
From: mgrojo <mgrojo@gmail.com>
Date: Sun, 3 Nov 2019 20:22:27 +0100
Subject: [PATCH] Disable entire toolbars and frames in Table Browser

The Show/Hide Style Toolbar button and the find bar buttons were not
being disabled when the database was closed. The Find Next button even made
the application crashed in that state.

This change ensures we don't forget to disable new buttons in the toolbars
and frames of the Table Browser.

The layout where the navigation buttons are inserted does not, apparently,
disable the children widget, so cannot be used in the same way.

See related issues #1976 and #1608.
---
 src/TableBrowser.cpp | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/TableBrowser.cpp b/src/TableBrowser.cpp
index 21fdfabe..be876d0b 100644
--- a/src/TableBrowser.cpp
+++ b/src/TableBrowser.cpp
@@ -361,19 +361,17 @@ QModelIndex TableBrowser::currentIndex() const
 
 void TableBrowser::setEnabled(bool enable)
 {
+    ui->browseToolbar->setEnabled(enable);
+    ui->editGlobalFilter->setEnabled(enable);
+    ui->formatFrame->setEnabled(enable);
+    ui->frameFind->setEnabled(enable);
+
     ui->buttonNext->setEnabled(enable);
     ui->buttonPrevious->setEnabled(enable);
     ui->buttonBegin->setEnabled(enable);
     ui->buttonEnd->setEnabled(enable);
     ui->buttonGoto->setEnabled(enable);
     ui->editGoto->setEnabled(enable);
-    ui->actionSaveFilterAsPopup->setEnabled(enable);
-    ui->actionClearFilters->setEnabled(enable);
-    ui->actionClearSorting->setEnabled(enable);
-    ui->actionRefresh->setEnabled(enable);
-    ui->actionPrintTable->setEnabled(enable);
-    ui->editGlobalFilter->setEnabled(enable);
-    ui->actionFind->setEnabled(enable);
 
     updateInsertDeleteRecordButton();
 }