visidata 3.1.1__py3-none-any.whl → 3.2__py3-none-any.whl

visidata/selection.py

@@ -20,9 +20,10 @@ def isSelected(self, row):
 
 @Sheet.api
 @asyncthread
-def toggle(self, rows):
+def toggle(self, rows, add_undo=True):
     'Toggle selection of given *rows*.  Async.'
-    self.addUndoSelection()
+    if add_undo:
+        self.addUndoSelection()
     for r in Progress(rows, 'toggling', total=len(rows)):
         if self.isSelected(r):  #1671
             self.unselectRow(r)
@@ -30,17 +31,24 @@ def toggle(self, rows):
             self.selectRow(r)
 
 
+@Sheet.before
+def beforeLoad(self):
+    self._selectedRows.clear()
+
+
 @Sheet.api
-def select_row(self, row):
+def select_row(self, row, add_undo=True):
     'Add single *row* to set of selected rows.'
-    self.addUndoSelection()
+    if add_undo:
+        self.addUndoSelection()
     self.selectRow(row)
 
 
 @Sheet.api
-def toggle_row(self, row):
+def toggle_row(self, row, add_undo=True):
     'Toggle selection of given *row*.'
-    self.addUndoSelection()
+    if add_undo:
+        self.addUndoSelection()
     if self.isSelected(row):
         self.unselectRow(row)
     else:
@@ -48,9 +56,10 @@ def toggle_row(self, row):
 
 
 @Sheet.api
-def unselect_row(self, row):
+def unselect_row(self, row, add_undo=True):
     'Remove single *row* from set of selected rows.'
-    self.addUndoSelection()
+    if add_undo:
+        self.addUndoSelection()
     self.unselectRow(row) or vd.warning('row not selected')
 
 
@@ -77,9 +86,10 @@ def clearSelected(self):
 
 @Sheet.api
 @asyncthread
-def select(self, rows, status=True, progress=True):
-    "Add *rows* to set of selected rows. Async. Don't show progress if *progress* is False; don't show status if *status* is False."
-    self.addUndoSelection()
+def select(self, rows, status=True, progress=True, add_undo=True):
+    "Add *rows* to set of selected rows. Async. Don't show progress if *progress* is False; don't show status if *status* is False. If *add_undo* is False, do not add an undo selection function to the undo history; useful for lowering memory consumption when caller is changing a large batch of selects in one command."
+    if add_undo:
+        self.addUndoSelection()
     before = self.nSelectedRows
     if self.options.bulk_select_clear:
         self.clearSelected()
@@ -94,9 +104,10 @@ def select(self, rows, status=True, progress=True):
 
 @Sheet.api
 @asyncthread
-def unselect(self, rows, status=True, progress=True):
-    "Remove *rows* from set of selected rows. Async. Don't show progress if *progress* is False; don't show status if *status* is False."
-    self.addUndoSelection()
+def unselect(self, rows, status=True, progress=True, add_undo=True):
+    "Remove *rows* from set of selected rows. Async. Don't show progress if *progress* is False; don't show status if *status* is False. If *add_undo* is False, do not add an undo unselection function to the undo history; useful for lowering memory consumption when caller is changing a large batch of selects in one command."
+    if add_undo:
+        self.addUndoSelection()
     before = self.nSelectedRows
     for r in (Progress(rows, 'unselecting') if progress else rows):
         self.unselectRow(r)
@@ -191,10 +202,10 @@ Sheet.addCommand('\\', 'unselect-col-regex', 'unselectByIdx(searchInputRegex("un
 Sheet.addCommand('g|', 'select-cols-regex', 'selectByIdx(searchInputRegex("select", columns="visibleCols"))', 'select rows matching regex in any visible column')
 Sheet.addCommand('g\\', 'unselect-cols-regex', 'unselectByIdx(searchInputRegex("unselect", columns="visibleCols"))', 'unselect rows matching regex in any visible column')
 
-Sheet.addCommand(',', 'select-equal-cell', 'select(gatherBy(lambda r,c=cursorCol,v=cursorDisplay: c.getDisplayValue(r) == v), progress=False)', 'select rows matching current cell in current column')
-Sheet.addCommand('g,', 'select-equal-row', 'select(gatherBy(lambda r,currow=cursorRow,vcols=visibleCols: all([c.getDisplayValue(r) == c.getDisplayValue(currow) for c in vcols])), progress=False)', 'select rows matching current row in all visible columns')
-Sheet.addCommand('z,', 'select-exact-cell', 'select(gatherBy(lambda r,c=cursorCol,v=cursorTypedValue: c.getTypedValue(r) == v), progress=False)', 'select rows matching current cell in current column')
-Sheet.addCommand('gz,', 'select-exact-row', 'select(gatherBy(lambda r,currow=cursorRow,vcols=visibleCols: all([c.getTypedValue(r) == c.getTypedValue(currow) for c in vcols])), progress=False)', 'select rows matching current row in all visible columns')
+Sheet.addCommand(',', 'select-equal-cell', 'select(gatherBy(lambda r,c=cursorCol,v=cursorDisplay: c.getDisplayValue(r) == v), progress=False)', 'select rows matching current cell displayed value in current column')
+Sheet.addCommand('g,', 'select-equal-row', 'select(gatherBy(lambda r,currow=cursorRow,vcols=visibleCols: all([c.getDisplayValue(r) == c.getDisplayValue(currow) for c in vcols])), progress=False)', 'select rows matching displayed values in current row in all visible columns')
+Sheet.addCommand('z,', 'select-exact-cell', 'select(gatherBy(lambda r,c=cursorCol,v=cursorTypedValue: c.getTypedValue(r) == v), progress=False)', 'select rows matching current cell typed value in current column')
+Sheet.addCommand('gz,', 'select-exact-row', 'select(gatherBy(lambda r,currow=cursorRow,vcols=visibleCols: all([c.getTypedValue(r) == c.getTypedValue(currow) for c in vcols])), progress=False)', 'select rows matching typed values in current row in all visible columns')
 
 Sheet.addCommand('z|', 'select-expr', 'expr=inputExpr("select by expr: "); select(gatherBy(lambda r, sheet=sheet, expr=expr, curcol=cursorCol: sheet.evalExpr(expr, r, curcol=curcol)), progress=False)', 'select rows matching Python expression in any visible column')
 Sheet.addCommand('z\\', 'unselect-expr', 'expr=inputExpr("unselect by expr: "); unselect(gatherBy(lambda r, sheet=sheet, expr=expr, curcol=cursorCol: sheet.evalExpr(expr, r, curcol=curcol)), progress=False)', 'unselect rows matching Python expression in any visible column')

visidata/settings.py

@@ -446,7 +446,7 @@ def loadConfigAndPlugins(vd, args=AttrDict()):
     # autoload installed plugins first
     args_plugins_autoload = args.plugins_autoload if 'plugins_autoload' in args else True
     if not args.nothing and args_plugins_autoload and vd.options.plugins_autoload:
-        from importlib_metadata import entry_points  # a backport which supports < 3.8 https://github.com/pypa/twine/pull/732
+        from importlib.metadata import entry_points
         try:
             eps = entry_points()
             eps_visidata = eps.select(group='visidata.plugins') if 'visidata.plugins' in eps.groups else []
@@ -550,7 +550,7 @@ def setPersistentOptions(vd, **kwargs):
                 fp.write(f'options.{optname}={repr(optval)}\n')
 
 
-vd.option('visidata_dir', '~/.visidata/', 'directory to load and store additional files', sheettype=None)
+vd.option('visidata_dir', user_config_dir('visidata'), 'directory to load and store additional files', sheettype=None)
 
 BaseSheet.bindkey('^M', '^J')  # for windows ENTER
 

visidata/sheets.py

@@ -22,6 +22,7 @@ vd.option('disp_wrap_break_long_words', False, 'break words longer than column w
 vd.option('disp_wrap_replace_whitespace', False, 'replace whitespace with spaces in multiline')
 vd.option('disp_wrap_placeholder', '…', 'multiline string to indicate truncation')
 vd.option('disp_multiline_focus', True, 'only multiline cursor row')
+vd.option('color_multiline_bottom', '', 'color of bottom line of multiline rows')  #2715
 vd.option('color_aggregator', 'bold 255 white on 234 black', 'color of aggregator summary on bottom row')
 
 
@@ -299,24 +300,43 @@ class TableSheet(BaseSheet):
         self.setKeys(self.columns[:self.nKeys])
 
     def loader(self):
-        'Reset rows and sync load ``source`` via iterload.  Overrideable.'
-        self.rows = []
+        'Reset rows and sync load ``source`` via iterload.  Overridable.'
         try:
-            with vd.Progress(gerund='loading', total=0):
-                for i, r in enumerate(self.iterload()):
-                    if self.precious and i > self.options.max_rows:
-                        break
-                    self.addRow(r)
+            for r in self._iterloader():
+                pass
         except FileNotFoundError:
             return  # let it be a blank sheet without error
 
+    def _iterloader(self):
+        self.rows = []
+        with vd.Progress(gerund='loading', total=0):
+            max_rows = self.options.max_rows
+            for i, r in enumerate(self.iterload()):
+                if self.precious and i >= max_rows:
+                    break
+                self.addRow(r)
+                yield r
+
     def iterload(self):
         'Generate rows from ``self.source``.  Override in subclass.'
         if False:
             yield vd.fail('no iterload for this loader yet')
 
+    def loadStart(self):
+        self.loaditer = self._iterloader()
+
+    def loadSome(self):
+        if not self.loaditer:
+            return False
+        try:
+            next(self.loaditer)
+            return True
+        except StopIteration:
+            self.loaditer = None
+            return False
+
     def afterLoad(self):
-        'hook for after loading has finished.  Overrideable (be sure to call super).'
+        'hook for after loading has finished.  Overridable (be sure to call super).'
         # if an ordering has been specified, sort the sheet
         if self._ordering:
             vd.sync(self.sort())
@@ -508,6 +528,9 @@ class TableSheet(BaseSheet):
         'Raw value at current row and column.'
         return self.cursorCol.getValue(self.cursorRow)
 
+    def getTypedRow(self, rownum):
+        return [c.getTypedValue(self.rows[rownum]) for c in self.availCols]
+
     @property
     def statusLine(self):
         'Position of cursor and bounds of current sheet.'
@@ -576,7 +599,10 @@ class TableSheet(BaseSheet):
         index = 0
         ccol = self.cursorCol
         if ccol and not ccol.keycol:
-            index = self.columns.index(ccol)+1
+            try:
+                index = self.columns.index(ccol)+1
+            except ValueError: # when all columns are hidden, the one column shown is not in self.columns
+                index = 0
 
         self.addColumn(*cols, index=index)
         firstnewcol = [c for c in cols if not c.hidden][0]
@@ -604,13 +630,6 @@ class TableSheet(BaseSheet):
         for col in cols:
             col.keycol = 0
 
-    def toggleKeys(self, cols):
-        for col in cols:
-            if col.keycol:
-                self.unsetKeys([col])
-            else:
-                self.setKeys([col])
-
     def rowkey(self, row):
         'Return tuple of the key for *row*.'
         return tuple(c.getTypedValue(row) for c in self.keyCols)
@@ -664,12 +683,13 @@ class TableSheet(BaseSheet):
                     continue
 
                 cur_x, cur_w = self._visibleColLayout[self.cursorVisibleColIndex]
-                if cur_x+cur_w < self.windowWidth:  # current columns fit entirely on screen
+                if cur_x+cur_w < self.windowWidth-1:  # current columns fit entirely on screen
                     break
                 self.leftVisibleColIndex += 1  # once within the bounds, walk over one column at a time
 
     def calcColLayout(self):
         'Set right-most visible column, based on calculation.'
+        vd.clearCaches()
         minColWidth = dispwidth(self.options.disp_more_left)+dispwidth(self.options.disp_more_right)+2
         sepColWidth = dispwidth(self.options.disp_column_sep)
         winWidth = self.windowWidth
@@ -701,7 +721,8 @@ class TableSheet(BaseSheet):
 
             width = max(width, 1)
             if col in self.keyCols or vcolidx >= self.leftVisibleColIndex:  # visible columns
-                self._visibleColLayout[vcolidx] = [x, min(width, self.windowWidth-x)]
+                #subtract 1 character of empty space from windowWidth, for the margin to the right of the sheet
+                self._visibleColLayout[vcolidx] = [x, max(min(width, self.windowWidth-x-1), 1)]
                 return width
 
 
@@ -744,7 +765,7 @@ class TableSheet(BaseSheet):
                 clipdraw(scr, y+i, x, name, hdrcattr, w=colwidth)
             vd.onMouse(scr, x, y+i, colwidth, 1, BUTTON3_RELEASED='rename-col')
 
-            if C and x+colwidth+len(C) < self.windowWidth and y+i < self.windowHeight:
+            if C and x+colwidth+dispwidth(C) < self.windowWidth and y+i < self.windowHeight:
                 scr.addstr(y+i, x+colwidth, C, sepcattr.attr)
 
         clipdraw(scr, y+h-1, min(x+colwidth, self.windowWidth-1)-dispwidth(T), T, hdrcattr)
@@ -777,6 +798,8 @@ class TableSheet(BaseSheet):
         'Return dict of aggname -> list of cols with that aggregator.'
         allaggs = collections.defaultdict(list) # aggname -> list of cols with that aggregator
         for vcolidx, (x, colwidth) in sorted(self._visibleColLayout.items()):
+            if vcolidx >= len(self.availCols):
+                break  #2607 #2763
             col = self.availCols[vcolidx]
             if not col.hidden:
                 for aggr in col.aggregators:
@@ -935,10 +958,12 @@ class TableSheet(BaseSheet):
                 colseps = [topsep] + [midsep]*(height-2) + [botsep]
                 endseps = [endtopsep] + [endmidsep]*(height-2) + [endbotsep]
                 keyseps = [keytopsep] + [keymidsep]*(height-2) + [keybotsep]
+                color_multiline_bottom = colors.get_color('color_multiline_bottom', 2)
             else:
                 colseps = [colsep]
                 endseps = [endsep]
                 keyseps = [keysep]
+                color_multiline_bottom = 0
 
             for vcolidx, (col, cellval, lines) in displines.items():
                     if vcolidx not in self._visibleColLayout:
@@ -957,6 +982,7 @@ class TableSheet(BaseSheet):
 
                     cattr = self._colorize(col, row, cellval)
                     cattr = update_attr(cattr, basecellcattr)
+                    bottomcattr = update_attr(cattr, color_multiline_bottom) if height > 1 else cattr
 
                     note = getattr(cellval, 'note', None)
                     notewidth = 1 if note else 0
@@ -984,10 +1010,10 @@ class TableSheet(BaseSheet):
                         for attr, text in chunks:
                             prechunks.append((attr, text[hoffset:]))
 
-                        clipdraw_chunks(scr, y, x, prechunks, cattr, w=colwidth-notewidth)
+                        clipdraw_chunks(scr, y, x, prechunks, cattr if i < height-1 else bottomcattr, w=colwidth-notewidth)
                         vd.onMouse(scr, x, y, colwidth, 1, BUTTON3_RELEASED='edit-cell')
 
-                        if sepchars and x+colwidth+dispwidth(sepchars) <= self.windowWidth:
+                        if sepchars and x+colwidth+dispwidth(sepchars) <= self.windowWidth-1:
                             scr.addstr(y, x+colwidth, sepchars, sepcattr.attr)
 
             for notefunc in vd.rowNoters:
@@ -1048,8 +1074,9 @@ class SequenceSheet(Sheet):
 
         self.rows = []
         # add the rest of the rows
+        max_rows = self.options.max_rows
         for i, r in enumerate(vd.Progress(itsource, gerund='loading', total=0)):
-            if self.precious and i > self.options.max_rows:
+            if self.precious and i >= max_rows:
                 break
             self.addRow(r)
 
@@ -1136,8 +1163,8 @@ def confirmQuit(vs, verb='quit'):
 def preloadHook(sheet):
     'Override to setup for reload().'
     sheet.confirmQuit('reload')
-
     sheet.hasBeenModified = False
+    sheet.calcColLayout()
 
 
 @VisiData.api
@@ -1189,7 +1216,8 @@ BaseSheet.init('pane', lambda: 1)
 BaseSheet.addCommand('^R', 'reload-sheet', 'preloadHook(); reload()', 'Reload current sheet')
 Sheet.addCommand('', 'show-cursor', 'status(statusLine)', 'show cursor position and bounds of current sheet on status line')
 
-Sheet.addCommand('!', 'key-col', 'toggleKeys([cursorCol])', 'toggle current column as a key column')
+Sheet.addCommand('!', 'key-col', 'exec_longname("key-col-off") if cursorCol.keycol else exec_longname("key-col-on")', 'toggle current column as a key column', replay=False)
+Sheet.addCommand('', 'key-col-on', 'setKeys([cursorCol])', 'set current column as a key column')
 Sheet.addCommand('z!', 'key-col-off', 'unsetKeys([cursorCol])', 'unset current column as a key column')
 
 Sheet.addCommand('e', 'edit-cell', 'cursorCol.setValues([cursorRow], editCell(cursorVisibleColIndex)) if not (cursorRow is None) else fail("no rows to edit")', 'edit contents of current cell')

visidata/shell.py

@@ -22,7 +22,7 @@ vd.option('dir_hidden', False, 'load hidden files on DirSheet')
 @VisiData.api
 def guess_dir(vd, p):
     if p.is_dir():
-        return dict(filetype='dir')
+        return dict(filetype='dir', _likelihood=10)
 
 
 @VisiData.lazy_property
@@ -133,7 +133,7 @@ class DirSheet(Sheet):
         Column('filetype', width=0, cache='async', getter=lambda col,row: subprocess.Popen(['file', '--brief', row], stdout=subprocess.PIPE, stderr=subprocess.PIPE).communicate()[0].strip()),
     ]
     nKeys = 2
-    _ordering = [('modtime', True)]  # sort by reverse modtime initially
+    _ordering = [('modtime', True), ('filename', False)]  # sort by reverse modtime initially
 
     @staticmethod
     def colorOwner(sheet, col, row, val):

visidata/sidebar.py

@@ -130,7 +130,7 @@ def drawSidebar(vd, scr, sheet):
 def drawSidebarText(sheet, scr, text:Union[None,str,'HelpPane'], title:str='', overflowmsg:str='', bottommsg:str=''):
     scrh, scrw = scr.getmaxyx()
     maxw = sheet.options.disp_sidebar_width or scrw//2
-    maxh = sheet.options.disp_sidebar_height or scrh-2
+    maxh = sheet.options.disp_sidebar_height or max(scrh-2, 1)
 
     cattr = colors.get_color('color_sidebar')
 
@@ -158,6 +158,7 @@ def drawSidebarText(sheet, scr, text:Union[None,str,'HelpPane'], title:str='', o
         if lines:
             maxlinew = max(maxlinew, max(dispwidth(textonly, maxwidth=maxw) for line, textonly in lines))
         winh = min(maxh, len(lines)+2)
+    winh = max(winh, 1)
 
     titlew = dispwidth(title)
 
@@ -166,6 +167,7 @@ def drawSidebarText(sheet, scr, text:Union[None,str,'HelpPane'], title:str='', o
     maxlinew = max(maxlinew, titlew)
     winw = min(maxw, maxlinew+4)
     x, y, w, h = scrw-winw-1, scrh-winh-1, winw, winh
+    y = max(y, 0)
 
     sidebarscr = vd.subwindow(scr, x, y, w, h)
 
@@ -191,7 +193,7 @@ def drawSidebarText(sheet, scr, text:Union[None,str,'HelpPane'], title:str='', o
     if bottommsg:
         clipdraw(sidebarscr, h-1, winw-dispwidth(bottommsg)-4, '|'+bottommsg+'|', cattr)
 
-    sidebarscr.refresh()
+    sidebarscr.noutrefresh()
 
 
 @VisiData.api

visidata/sort.py

@@ -1,15 +1,32 @@
 from copy import copy
-from visidata import vd, asyncthread, Progress, Sheet, options, UNLOADED
+from visidata import vd, asyncthread, Progress, Sheet, Column, options, UNLOADED, ColumnsSheet
+import re
+
+cmdlog_col_prefix='\u241f'  #string ␟ to mark the start of column info in an ordering string
 
 @Sheet.api
-def orderBy(sheet, *cols, reverse=False):
-    'Add *cols* to internal ordering and re-sort the rows accordingly.  Pass *reverse* as True to order these *cols* descending.  Pass empty *cols* (or cols[0] of None) to clear internal ordering.'
+def orderBy(sheet, *cols, reverse=False, change_column=False, save_cmd_input=True):
+    '''Add *cols* to internal ordering and re-sort the rows accordingly.
+    Pass *reverse* as True to order these *cols* descending.
+    Pass empty *cols* (or cols[0] of None) to clear internal ordering.
+    Set *change_column* to True to change the sort status of a single column: add/remove/invert it.
+    When changing a column, *cols* must have length 1. Sort columns that had higher priority are unchanged. Lower-priority columns are removed.
+    If *change_column* is False, *cols* will be added to the existing ordering.
+    If *save_cmd_input* is True, the full ordering that results will be saved in the cmdlog for future replay in the 'input' parameter.
+    '''
+
     if options.undo:
         vd.addUndo(setattr, sheet, '_ordering', copy(sheet._ordering))
-        if sheet._ordering:
-            vd.addUndo(sheet.sort)
-        else:
-            vd.addUndo(setattr, sheet, 'rows', copy(sheet.rows))
+        vd.addUndo(setattr, sheet, 'rows', copy(sheet.rows))
+
+    # for replay, read the full column ordering from the cmdlog input parameter  #2688
+    input = vd.getLastArgs()
+    if input:
+        sheet._ordering = order_from_string(sheet, input)
+        sheet.sort()
+        if save_cmd_input:
+            vd.activeCommand.input = order_string(sheet)
+        return
 
     do_sort = False
     if not cols or cols[0] is None:
@@ -17,12 +34,21 @@ def orderBy(sheet, *cols, reverse=False):
         cols = cols[1:]
         do_sort = True
 
-    for c in cols:
-        sheet._ordering.append((c, reverse))
+    if change_column:
+        if len(cols) > 1:
+            vd.fail('sort order edit must only be applied to a single column')
+        new_ordering = edit_ordering(sheet._ordering, cols[0], reverse)
+        sheet._ordering = new_ordering
         do_sort = True
+    else:
+        for c in cols:
+            sheet._ordering.append((c, reverse))
+            do_sort = True
 
     if do_sort:
         sheet.sort()
+    if save_cmd_input:
+        vd.activeCommand.input = order_string(sheet)
 
 class Reversor:
     def __init__(self, obj):
@@ -34,7 +60,43 @@ class Reversor:
     def __lt__(self, other):
         return other.obj < self.obj
 
+def order_string(sheet):
+    sheet._ordering = sheet.ordering  #converts any ambiguous colname strings to unambiguous Column objects
+    ret = ''.join([cmdlog_col_prefix+('>' if reverse else '<') + str(col.name) for col, reverse in sheet._ordering])
+    return ret
 
+def order_from_string(sheet, s):
+    instructions = re.split(cmdlog_col_prefix + '(?=[<>])', s)[1:]
+    ordering = []
+    for instr in instructions:
+        c = sheet.column(instr[1:])
+        if instr[0] == '<':
+            reverse = False
+        elif instr[0] == '>':
+            reverse = True
+        ordering.append((c, reverse))
+    return ordering
+
+def edit_ordering(ordering, col, reverse):
+    '''Return a modified ordering based on editing a single column *col*:   add it, remove it, or flip its direction.
+    Columns after *col* in the ordering (with lower sort priority) are also removed from the ordering.
+    *ordering* is a list of tuples:  (Column, boolean), where the boolean defines the sort direction.
+    '''
+    new_ordering = []
+    # handle changes to status of columns that are already in the ordering:  add/remove/flip
+    changed = False
+    for c, old_reverse in ordering:
+        if c is col:
+            if reverse != old_reverse: # reverse the column's sort direction
+                new_ordering.append((c, reverse))
+            # if the sort direction is unchanged, remove the column from the ordering
+            changed = True
+            # columns after the edited column will be dropped from the ordering
+            break
+        new_ordering.append((c, old_reverse))
+    if not changed:
+        new_ordering.append((col, reverse))
+    return new_ordering
 
 @Sheet.cached_property
 def ordering(sheet) -> 'list[tuple[Column, bool]]':
@@ -76,6 +138,20 @@ def sort(self):
         vd.warning('sort incomplete due to TypeError; change column type')
         vd.exceptionCaught(e, status=False)
 
+ColumnsSheet.columns += [
+        Column('sortorder',
+            type=int,
+            getter=lambda c,r: _sort_order(c, r),
+            help='sort priority and direction in source sheet')
+]
+
+def _sort_order(col, srccol):
+    sort_cols = [(n+1, reverse) for n, (c, reverse) in enumerate(srccol.sheet.ordering) if c is srccol]
+    if not sort_cols:
+        return None
+    n, reverse = sort_cols[0]
+    return -n if reverse else n
+
 
 # replace existing sort criteria
 Sheet.addCommand('[', 'sort-asc', 'orderBy(None, cursorCol)', 'sort ascending by current column; replace any existing sort criteria')
@@ -84,8 +160,10 @@ Sheet.addCommand('g[', 'sort-keys-asc', 'orderBy(None, *keyCols)', 'sort ascendi
 Sheet.addCommand('g]', 'sort-keys-desc', 'orderBy(None, *keyCols, reverse=True)', 'sort descending by all key columns; replace any existing sort criteria')
 
 # add to existing sort criteria
-Sheet.addCommand('z[', 'sort-asc-add', 'orderBy(cursorCol)', 'sort ascending by current column; add to existing sort criteria')
-Sheet.addCommand('z]', 'sort-desc-add', 'orderBy(cursorCol, reverse=True)', 'sort descending by current column; add to existing sort criteria')
+Sheet.addCommand('', 'sort-asc-add', 'orderBy(cursorCol)', 'sort ascending by current column; add to existing sort criteria')
+Sheet.addCommand('', 'sort-desc-add', 'orderBy(cursorCol, reverse=True)', 'sort descending by current column; add to existing sort criteria')
+Sheet.addCommand('z[', 'sort-asc-change', 'orderBy(cursorCol, change_column=True)', 'sort ascending by current column; keep higher priority sort criteria')
+Sheet.addCommand('z]', 'sort-desc-change', 'orderBy(cursorCol, reverse=True, change_column=True)', 'sort descending by current column; keep higher priority sort criteria')
 Sheet.addCommand('gz[', 'sort-keys-asc-add', 'orderBy(*keyCols)', 'sort ascending by all key columns; add to existing sort criteria')
 Sheet.addCommand('gz]', 'sort-keys-desc-add', 'orderBy(*keyCols, reverse=True)', 'sort descending by all key columns; add to existing sort criteria')
 

visidata/statusbar.py

@@ -8,7 +8,7 @@ import curses
 import sys
 
 import visidata
-from visidata import vd, VisiData, BaseSheet, Sheet, ColumnItem, Column, RowColorizer, options, colors, wrmap, clipdraw, ExpectedException, update_attr, dispwidth, ColorAttr
+from visidata import vd, VisiData, BaseSheet, Sheet, ColumnItem, Column, RowColorizer, options, colors, wrmap, clipdraw, ExpectedException, update_attr, dispwidth, ColorAttr, clipstr_middle
 
 
 
@@ -31,11 +31,6 @@ vd.theme_option('color_highlight_status', 'black on green', 'color of highlighte
 
 BaseSheet.init('longname', lambda: '')
 
-def fitWithin(s, n=10):
-    if len(s) > n:
-        return s[:n//2-1] + '…' + s[-n//2+1:]
-    return s
-
 @BaseSheet.property
 def ancestors(sheet):
     if isinstance(sheet.source, BaseSheet):
@@ -53,14 +48,16 @@ def sheetlist(sheet):
 
     sheetnames = []
     for vs in sheets:
+        if not vs.precious:  #2573
+            continue
         if isinstance(vs, BaseSheet):
             shortcut = ' '
             if vs.shortcut in '1 2 3 4 5 6 7 8 9 10'.split():
-                shortcut = vs.shortcut[-1] + '›'
+                shortcut = vs.shortcut[-1] + vs.icon
             if vs is vd.sheet:
                 sheetnames.append(f'[:menu_active]{shortcut}{vs.name}[:]')
             else:
-                sheetnames.append(f'[:onclick jump-sheet-{vs.shortcut}]' + fitWithin(f'{shortcut}{vs.name}', 20) + '[:]')
+                sheetnames.append(f'[:onclick jump-sheet-{vs.shortcut}]' + clipstr_middle(f'{shortcut}{vs.name}', 20)[0] + '[:]')
         else:
             sheetnames.append(vs)
 
@@ -226,12 +223,16 @@ def modifiedStatus(sheet):
     return ret
 
 
+@BaseSheet.property
+def selectedStatus(sheet):
+    return ''
+
+
 @Sheet.property
 def selectedStatus(sheet):
     if sheet.nSelectedRows:
         return f' [:selected_row][:onclick dup-selected]{sheet.options.disp_selected_note}{sheet.nSelectedRows}[/][/] '
 
-
 @VisiData.api
 def drawRightStatus(vd, scr, vs):
     'Draw right side of status bar.  Return length displayed.'