Move KeyString's enable/disable logic to separate class

qutebrowser/mainwindow/mainwindow.py

@@ -497,7 +497,7 @@ def _connect_signals(self):
 
         # commands
         mode_manager.keystring_updated.connect(
-            self.status.keystring.on_keystring_updated)
+            self.status.keystring.widget.on_keystring_updated)
         self.status.cmd.got_cmd[str].connect(self._commandrunner.run_safely)
         self.status.cmd.got_cmd[str, int].connect(self._commandrunner.run_safely)
         self.status.cmd.returnPressed.connect(self.tabbed_browser.on_cmd_return_pressed)

qutebrowser/mainwindow/statusbar/bar.py

@@ -185,7 +185,7 @@ def __init__(self, *, win_id, private, parent=None):
         self.percentage = percentage.Percentage(widget=percentage.PercentageWidget())
         self.backforward = backforward.Backforward(widget=backforward.BackforwardWidget())
         self.tabindex = tabindex.TabIndex()
-        self.keystring = keystring.KeyString()
+        self.keystring = keystring.KeyString(widget=keystring.KeyStringWidget())
         self.prog = progress.Progress(widget=progress.ProgressWidget(self))
         self.clock = clock.Clock(widget=clock.ClockWidget())
         self._text_widgets = []
@@ -247,7 +247,7 @@ def _draw_widgets(self):
             # instances for all widgets.
             if isinstance(widget, (url.UrlText, backforward.Backforward,
                                    progress.Progress, percentage.Percentage,
-                                   clock.Clock)):
+                                   clock.Clock, keystring.KeyString)):
                 widget = widget.widget
 
             self._hbox.addWidget(widget)
@@ -278,7 +278,7 @@ def _clear_widgets(self):
         # Start with widgets hidden and show them when needed
         for widget in [self.url.widget, self.percentage.widget,
                        self.backforward.widget, self.tabindex,
-                       self.keystring, self.prog.widget, self.clock.widget, *self._text_widgets]:
+                       self.keystring.widget, self.prog.widget, self.clock.widget, *self._text_widgets]:
             assert isinstance(widget, QWidget)
             if widget in [self.prog.widget, self.backforward.widget]:
                 widget.enabled = False  # type: ignore[attr-defined]

qutebrowser/mainwindow/statusbar/keystring.py

@@ -10,10 +10,21 @@
 from qutebrowser.utils import usertypes
 
 
-class KeyString(textbase.TextBase):
+class KeyStringWidget(textbase.TextBase):
 
     """Keychain string displayed in the statusbar."""
 
     @pyqtSlot(usertypes.KeyMode, str)
     def on_keystring_updated(self, _mode, keystr):
         self.setText(keystr)
+
+
+class KeyString:
+    def __init__(self, widget: KeyStringWidget):
+        self.widget = widget
+
+    def enable(self):
+        self.widget.show()
+
+    def disable(self):
+        self.widget.hide()