diff options
| author | Benjamin Poulain <benjamin.poulain@nokia.com> | 2010-08-22 01:17:43 +0200 |
|---|---|---|
| committer | Benjamin Poulain <benjamin.poulain@nokia.com> | 2010-08-22 04:10:58 +0200 |
| commit | fe02d0962810b1ebca33c219274a5e3390de5286 (patch) | |
| tree | c922c970b2cc3039b41775abe1089e452f9ad515 /src/mainwindow.cpp | |
| parent | Restore bk icon on the right (diff) | |
| download | rekonq-fe02d0962810b1ebca33c219274a5e3390de5286.tar.xz | |
Make the FindBar a bit more robust
The FindBar was slots show() and hide() instead of the virtual method
QWidget::setVisible(). This can lead to subtle bugs, because the
behavior is different depending on the type of call (signal-slot
connection VS direct call on a pointer to QWidget) and the function
called (hide() is != than setVisible(false)).
Diffstat (limited to 'src/mainwindow.cpp')
| -rw-r--r-- | src/mainwindow.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp index 4f9d6dab..28a3bebf 100644 --- a/src/mainwindow.cpp +++ b/src/mainwindow.cpp @@ -370,6 +370,7 @@ void MainWindow::setupActions() findShortcut.setAlternate(Qt::Key_Slash); a->setShortcut(findShortcut); a->setCheckable(true); + a->setChecked(m_findBar->isVisible()); connect(m_findBar, SIGNAL(visibilityChanged(bool)), a, SLOT(setChecked(bool))); KStandardAction::findNext(this, SLOT(findNext()) , actionCollection()); |