From 2456b456ae72c7c56c03aaf131aef7a2e7ad011b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Touceda?= <chiiph@leap.se>
Date: Wed, 5 Jun 2013 11:58:17 -0300
Subject: Only show question mark when the previous check passed

---
 src/leap/gui/wizard.py | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/leap/gui/wizard.py b/src/leap/gui/wizard.py
index 405752ef..3cf1765b 100644
--- a/src/leap/gui/wizard.py
+++ b/src/leap/gui/wizard.py
@@ -389,10 +389,11 @@ class Wizard(QtGui.QWizard):
         if not passed:
             status = self.tr("<font color='red'><b>Non-existent "
                              "provider</b></font>")
+        else:
+            self.ui.lblHTTPS.setPixmap(self.QUESTION_ICON)
         self.ui.lblProviderSelectStatus.setText(status)
         self.ui.btnCheck.setEnabled(not passed)
         self.ui.lnProvider.setEnabled(not passed)
-        self.ui.lblHTTPS.setPixmap(self.QUESTION_ICON)
 
     def _https_connection(self, data):
         """
@@ -408,9 +409,10 @@ class Wizard(QtGui.QWizard):
             status = self.tr("<font color='red'><b>%s</b></font>") \
                 % (data[self._provider_bootstrapper.ERROR_KEY])
             self.ui.lblProviderSelectStatus.setText(status)
+        else:
+            self.ui.lblProviderInfo.setPixmap(self.QUESTION_ICON)
         self.ui.btnCheck.setEnabled(not passed)
         self.ui.lnProvider.setEnabled(not passed)
-        self.ui.lblProviderInfo.setPixmap(self.QUESTION_ICON)
 
     def _download_provider_info(self, data):
         """
@@ -451,7 +453,9 @@ class Wizard(QtGui.QWizard):
         Sets the status for the download of the CA certificate check
         """
         self._complete_task(data, self.ui.lblDownloadCaCert)
-        self.ui.lblCheckCaFpr.setPixmap(self.QUESTION_ICON)
+        passed = data[self._provider_bootstrapper.PASSED_KEY]
+        if passed:
+            self.ui.lblCheckCaFpr.setPixmap(self.QUESTION_ICON)
 
     def _check_ca_fingerprint(self, data):
         """
@@ -461,7 +465,9 @@ class Wizard(QtGui.QWizard):
         Sets the status for the CA fingerprint check
         """
         self._complete_task(data, self.ui.lblCheckCaFpr)
-        self.ui.lblCheckApiCert.setPixmap(self.QUESTION_ICON)
+        passed = data[self._provider_bootstrapper.PASSED_KEY]
+        if passed:
+            self.ui.lblCheckApiCert.setPixmap(self.QUESTION_ICON)
 
     def _check_api_certificate(self, data):
         """
-- 
cgit v1.2.3