Skip to content

Commit

Permalink
Remove unused _text_widgets
Browse files Browse the repository at this point in the history
Looks like `_text_widgets` is on longer necessary. I hope this is the result of my
glorious refactoring skills and not just me being too tired to realize I am missing
something.
  • Loading branch information
pylbrecht committed Sep 8, 2024
1 parent 0d89940 commit 6194126
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions qutebrowser/mainwindow/statusbar/bar.py
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,6 @@ def __init__(self, *, win_id, private, parent=None):
self.keystring = keystring.KeyString(widget=textbase.TextBaseWidget())
self.prog = progress.Progress(widget=progress.ProgressWidget(self))
self.clock = clock.Clock(widget=clock.ClockWidget())
self._text_widgets = []
self._draw_widgets()

config.instance.changed.connect(self._on_config_changed)
Expand Down Expand Up @@ -276,10 +275,6 @@ def _draw_widgets(self):
for segment in config.val.statusbar.widgets:
item = _create_item_from_config(segment, tab=tab, parent=self)

# FIXME(pylbrecht): get rid of this special case
if segment.startswith("text:"):
self._text_widgets.append(item)

self._hbox.addWidget(item.widget)

item.enable()
Expand All @@ -293,7 +288,6 @@ def _clear_widgets(self):
for widget in _ITEMS.values():
widget.disable()
self._hbox.removeWidget(widget.widget)
self._text_widgets.clear()

@pyqtSlot()
def maybe_hide(self):
Expand Down

0 comments on commit 6194126

Please sign in to comment.