# HG changeset patch # User Andre Heinecke # Date 1413202887 -7200 # Node ID 135891f1c8f44ad11d70ffd41baa95b23021c185 # Parent 60e481aa75cafa9698337919d006ba4e1a8e2f82 (issue154) Do not force the details button to be active if a certificate changes This reuses the activate function to reduce redundancy. diff -r 60e481aa75ca -r 135891f1c8f4 ui/mainwindow.cpp --- a/ui/mainwindow.cpp Mon Oct 13 14:13:05 2014 +0200 +++ b/ui/mainwindow.cpp Mon Oct 13 14:21:27 2014 +0200 @@ -1223,9 +1223,9 @@ .arg(mUpdatesNew->selectedCertCount()) .arg(mUpdatesNew->certificates().size()) + ""); - mUpdatesDetailsNew->show(); - mUpdatesDetailsNew->setIcon(QIcon(":/img/dialog-information_16px.png")); - mUpdatesDetailsNew->setToolTip(tr("Show details")); + if (!mUpdatesNew->isVisible()) { + activateDetailsButton(mUpdatesDetailsNew); + } mUpdatesNewCertificates->show(); } else { mUpdatesDetailsNew->hide(); @@ -1240,9 +1240,9 @@ .arg(mUpdatesRemove->certificates().size()) + ""); mUpdatesRemoveCertificates->show(); - mUpdatesDetailsRemove->setIcon(QIcon(":/img/dialog-information_16px.png")); - mUpdatesDetailsRemove->setToolTip(tr("Show details")); - mUpdatesDetailsRemove->show(); + if (!mUpdatesRemove->isVisible()) { + activateDetailsButton(mUpdatesDetailsRemove); + } } else { mUpdatesRemoveCertificates->hide(); mUpdatesDetailsRemove->hide(); @@ -1676,6 +1676,7 @@ btn->setToolTip(QObject::tr("Show details")); btn->setText(" " + QObject::tr("Details")); btn->setIcon(QIcon(":/img/dialog-information_16px.png")); + btn->show(); } void MainWindow::toggleUpdatesNew() {