commit 2cdee9afa60c8d72654a503a0975936b1e351add Author: Damian Johnson atagar@torproject.org Date: Wed Mar 16 10:22:34 2016 -0700
Drop Panel's get_parent() method
Only used in one spot, and a weird one at that. I want to get rid of this attribute entirely but it's still used within the Panel itself. --- nyx/panel/__init__.py | 7 ------- nyx/panel/header.py | 11 +++++------ 2 files changed, 5 insertions(+), 13 deletions(-)
diff --git a/nyx/panel/__init__.py b/nyx/panel/__init__.py index 287d756..176eea9 100644 --- a/nyx/panel/__init__.py +++ b/nyx/panel/__init__.py @@ -192,13 +192,6 @@ class Panel(object):
self.title_visible = is_visible
- def get_parent(self): - """ - Provides the parent used to create subwindows. - """ - - return self.parent - def set_visible(self, is_visible): """ Toggles if the panel is visible or not. diff --git a/nyx/panel/header.py b/nyx/panel/header.py index 3ee0e84..51245ee 100644 --- a/nyx/panel/header.py +++ b/nyx/panel/header.py @@ -44,21 +44,19 @@ class HeaderPanel(nyx.panel.Panel, threading.Thread):
self._vals = get_sampling()
+ self._last_width = 100 self._pause_condition = threading.Condition() self._halt = False # terminates thread if true self._reported_inactive = False
tor_controller().add_status_listener(self.reset_listener)
- def is_wide(self, width = None): + def is_wide(self): """ True if we should show two columns of information, False otherwise. """
- if width is None: - width = self.get_parent().getmaxyx()[1] - - return width >= MIN_DUAL_COL_WIDTH + return self._last_width >= MIN_DUAL_COL_WIDTH
def get_height(self): """ @@ -127,7 +125,8 @@ class HeaderPanel(nyx.panel.Panel, threading.Thread):
def draw(self, width, height): vals = self._vals # local reference to avoid concurrency concerns - is_wide = self.is_wide(width) + self._last_width = width + is_wide = self.is_wide()
# space available for content