eric6/WebBrowser/AdBlock/AdBlockDialog.py

changeset 8260
2161475d9639
parent 8218
7c09585bd960
--- a/eric6/WebBrowser/AdBlock/AdBlockDialog.py	Wed Apr 21 19:40:50 2021 +0200
+++ b/eric6/WebBrowser/AdBlock/AdBlockDialog.py	Thu Apr 22 18:02:47 2021 +0200
@@ -84,10 +84,11 @@
         from .AdBlockTreeWidget import AdBlockTreeWidget
         for subscription in self.__manager.subscriptions():
             tree = AdBlockTreeWidget(subscription, self.subscriptionsTabWidget)
-            if subscription.isEnabled():
-                icon = UI.PixmapCache.getIcon("adBlockPlus")
-            else:
-                icon = UI.PixmapCache.getIcon("adBlockPlusDisabled")
+            icon = (
+                UI.PixmapCache.getIcon("adBlockPlus")
+                if subscription.isEnabled() else
+                UI.PixmapCache.getIcon("adBlockPlusDisabled")
+            )
             self.subscriptionsTabWidget.addTab(
                 tree, icon, subscription.title())
         
@@ -230,17 +231,18 @@
         )
         for subscription in requiresSubscriptions:
             requiresTitles.append(subscription.title())
-        if requiresTitles:
-            message = self.tr(
+        message = (
+            self.tr(
                 "<p>Do you really want to remove subscription"
                 " <b>{0}</b> and all subscriptions requiring it?</p>"
                 "<ul><li>{1}</li></ul>").format(
                 self.__currentSubscription.title(),
                 "</li><li>".join(requiresTitles))
-        else:
-            message = self.tr(
+            if requiresTitles else
+            self.tr(
                 "<p>Do you really want to remove subscription"
                 " <b>{0}</b>?</p>").format(self.__currentSubscription.title())
+        )
         res = E5MessageBox.yesNo(
             self,
             self.tr("Remove Subscription"),

eric ide

mercurial