Merge pull request #2373 from midichef/side_colors

[color-] make status colors more readable

visidata/guide.py

@@ -115,7 +115,7 @@ class OptionHelpGetter:
  'For easy and consistent formatting in sidebars and helpstrings, use {help.options.opt_name}.'
  def __getattr__(self, optname):
  opt = vd.options._get(optname, 'default')
- return f'[:onclick options-sheet {optname}][:longname]{optname}[/][/]: {opt.helpstr} (default: {opt.value})'
+ return f'[:onclick options-sheet {optname}][:longname_guide]{optname}[/][/]: {opt.helpstr} (default: {opt.value})'
 
 
 class CommandHelpGetter:
@@ -149,7 +149,7 @@ def __getitem__(self, k):
  binding += f'[:45] {inputtype}[/]'
 
  helpstr = cmd.helpstr
- return f'[:keys]{binding}[/] ([:longname]{longname}[/]) to {helpstr}'
+ return f'[:keys]{binding}[/] [:longname_guide]{longname}[/] to {helpstr}'
 
 
 class GuideSheet(Sheet):

visidata/sidebar.py

@@ -136,7 +136,7 @@ def drawSidebarText(sheet, scr, text:Union[None,str,'HelpPane'], title:str='', o
  x = max(0, w-titlew-6)
  clipdraw(sidebarscr, 0, x, f"|[:sidebar_title] {title} [:]|", cattr, w=titlew+4)
  if bottommsg:
- clipdraw(sidebarscr, h-1, winw-dispwidth(bottommsg)-4, '|'+bottommsg+'|[:]', cattr)
+ clipdraw(sidebarscr, h-1, winw-dispwidth(bottommsg)-4, '|'+bottommsg+'|', cattr)
 
  sidebarscr.refresh()
 

visidata/statusbar.py

@@ -11,13 +11,14 @@
 from visidata import vd, VisiData, BaseSheet, Sheet, ColumnItem, Column, RowColorizer, options, colors, wrmap, clipdraw, ExpectedException, update_attr, dispwidth, ColorAttr
 
 
-vd.option('disp_rstatus_fmt', '{sheet.threadStatus} {sheet.keystrokeStatus} [:longname]{sheet.longname}[/] {sheet.nRows:9d} {sheet.rowtype} {sheet.modifiedStatus}{sheet.selectedStatus}{vd.replayStatus}', 'right-side status format string')
+vd.option('disp_rstatus_fmt', '{sheet.threadStatus} {sheet.keystrokeStatus} [:longname_status]{sheet.longname}[/] {sheet.nRows:9d} {sheet.rowtype} {sheet.modifiedStatus}{sheet.selectedStatus}{vd.replayStatus}', 'right-side status format string')
 vd.option('disp_status_fmt', '{sheet.sheetlist}| ', 'left-side status format string')
 vd.theme_option('disp_lstatus_max', 0, 'maximum length of left status line')
 vd.theme_option('disp_status_sep', '│', 'separator between statuses')
 
 vd.theme_option('color_keystrokes', 'bold white on 237', 'color of input keystrokes')
-vd.theme_option('color_longname', '6', 'color of command longnames')
+vd.theme_option('color_longname_guide', '6', 'color of command longnames')
+vd.theme_option('color_longname_status', 'white', 'color of command longnames')
 vd.theme_option('color_keys', 'bold', 'color of keystrokes in help')
 vd.theme_option('color_status', 'bold on 238', 'status line color')
 vd.theme_option('color_error', '202 1', 'error message color')

visidata/themes/ascii8.py

@@ -68,7 +68,8 @@
  color_top_status='underline',
  color_active_status='black on cyan',
  color_inactive_status='8 on black',
- color_working='green',
+ color_working='bold white',
+ color_longname_status='black on cyan',
 
  color_menu='black on cyan',
  color_menu_active='yellow on black',

visidata/themes/asciimono.py

@@ -48,9 +48,13 @@
  color_hidden_col='8',
  color_selected_row='',
  color_edit_cell='',
+ color_edit_unfocused='',
  color_graph_hidden='',
  color_graph_selected='bold',
  color_status_replay='',
+ color_currency_neg='',
+ color_match='',
+ color_cmdpalette='',
 
  color_graph_axis='bold',
  color_sidebar='reverse',
@@ -69,6 +73,12 @@
  color_active_status='reverse',
  color_inactive_status='8',
  color_working='',
+ color_longname='',
+ color_highlight_status='',
+ color_sidebar_title='',
+ color_heading='',
+ color_guide_unwritten='',
+ color_code='',
 
  color_menu='reverse',
  color_menu_active='',