diff options
| author | hut <hut@lavabit.com> | 2009-12-30 17:03:41 +0100 |
|---|---|---|
| committer | hut <hut@lavabit.com> | 2009-12-30 17:03:41 +0100 |
| commit | 328c656557583152ecb58a5083581f433ec31f14 (patch) | |
| tree | 6b5db052d156923f7782fad26dc26e21ce102ba6 /ranger | |
| parent | 316421545abeb50d5bd7d3ffdbab51f44fdda45e (diff) | |
| download | ranger-328c656557583152ecb58a5083581f433ec31f14.tar.gz | |
filelist widget optimization
instead of containing another pager widget, filelist now inherits the pagers properties for displaying the content of files
Diffstat (limited to 'ranger')
| -rw-r--r-- | ranger/gui/widgets/filelist.py | 35 | ||||
| -rw-r--r-- | ranger/gui/widgets/pager.py | 5 |
2 files changed, 16 insertions, 24 deletions
diff --git a/ranger/gui/widgets/filelist.py b/ranger/gui/widgets/filelist.py index 8d37b92a..4dd04ed1 100644 --- a/ranger/gui/widgets/filelist.py +++ b/ranger/gui/widgets/filelist.py @@ -1,10 +1,9 @@ """The FileList widget displays the contents of a directory or file.""" from . import Widget -from ..displayable import DisplayableContainer from .pager import Pager from ranger import log -class FileList(Widget, DisplayableContainer): +class FileList(Pager, Widget): main_display = False display_infostring = False scroll_begin = 0 @@ -13,14 +12,12 @@ class FileList(Widget, DisplayableContainer): tagged_marker = '*' def __init__(self, win, level): - DisplayableContainer.__init__(self, win) - self.pager = Pager(self.win) - self.add_obj(self.pager) + Pager.__init__(self, win) + Widget.__init__(self, win) self.level = level def resize(self, y, x, hei, wid): - DisplayableContainer.resize(self, y, x, hei, wid) - self.pager.resize(0, 0, hei, wid) + Widget.resize(self, y, x, hei, wid) def click(self, event): """Handle a MouseEvent""" @@ -75,15 +72,15 @@ class FileList(Widget, DisplayableContainer): from ranger.fsobject.file import File from ranger.fsobject.directory import Directory - self.pager.visible = False +# self.pager.visible = False if self.target is None: pass elif type(self.target) == File: + Pager.open(self) self._draw_file() elif type(self.target) == Directory: self._draw_directory() - - DisplayableContainer.draw(self) + Widget.draw(self) def _preview_this_file(self, target): return target.document and not self.settings.preview_files @@ -93,18 +90,20 @@ class FileList(Widget, DisplayableContainer): self.win.move(0, 0) if not self.target.accessible: self.win.addnstr("not accessible", self.wid) + Pager.close(self) return if not self._preview_this_file(self.target): + Pager.close(self) return try: f = open(self.target.path, 'r') except: - pass + Pager.close(self) else: - self.pager.visible = True - self.pager.set_source(f) + self.set_source(f) + Pager.draw(self) def _draw_directory(self): """Draw the contents of a directory""" @@ -121,16 +120,6 @@ class FileList(Widget, DisplayableContainer): self.target.sort_if_outdated() if not self.target.content_loaded: -# if self.target.force_load: -# self.target.stopped = False -# -# else: -# if not self.target.stopped: -# -# if maxdirsize is not None and self.target.accessible \ -# and self.target.size > maxdirsize: -# self.target.stopped = True -# maxdirsize = self.settings.max_dirsize_for_autopreview if not self.target.force_load and maxdirsize is not None \ and self.target.accessible \ diff --git a/ranger/gui/widgets/pager.py b/ranger/gui/widgets/pager.py index 1664cc4b..85871431 100644 --- a/ranger/gui/widgets/pager.py +++ b/ranger/gui/widgets/pager.py @@ -37,7 +37,10 @@ class Pager(Widget): starty=self.scroll_begin, startx=self.startx) for line, i in zip(line_gen, range(self.hei)): - self.addstr(i, 0, line) + try: + self.addstr(i, 0, line) + except TypeError: + pass def move(self, relative=0, absolute=None): i = self.scroll_begin |