ttygroup/boartty
Code Issues Proposed changes

Advance cursor on change list toggle

Browse Source 
When toggling any flag on a row in the change list, advance the
cursor to the next row.

Change-Id: I0d140748a1bccf97d3c3b9b132d76947caa6f820
This commit is contained in:
James E. Blair 2015-05-23 15:46:02 -07:00
parent 9cdfa07f91
commit 50aa0f9558
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
gertty/view/change_list.py
View File

@ -408,6 +408,12 @@ class ChangeListView(urwid.WidgetWrap):
self.log.debug("Set change %s to %s", change_key, hidden_str) self.log.debug("Set change %s to %s", change_key, hidden_str)
return ret return ret
def advance(self):
pos = self.listbox.focus_position
if pos < len(self.listbox.body)-1:
pos += 1
self.listbox.focus_position = pos
def keypress(self, size, key): def keypress(self, size, key):
r = super(ChangeListView, self).keypress(size, key) r = super(ChangeListView, self).keypress(size, key)
commands = self.app.config.keymap.getCommands(r) commands = self.app.config.keymap.getCommands(r)
@ -433,6 +439,7 @@ class ChangeListView(urwid.WidgetWrap):
# where we're not just popping a row from the list of unreviewed # where we're not just popping a row from the list of unreviewed
# changes. # changes.
self.refresh() self.refresh()
self.advance()
return None return None
if keymap.TOGGLE_HIDDEN in commands: if keymap.TOGGLE_HIDDEN in commands:
if not len(self.listbox.body): if not len(self.listbox.body):
@ -450,6 +457,7 @@ class ChangeListView(urwid.WidgetWrap):
# Just fall back on doing a full refresh if we're in a situation # Just fall back on doing a full refresh if we're in a situation
# where we're not just popping a row from the list of changes. # where we're not just popping a row from the list of changes.
self.refresh() self.refresh()
self.advance()
return None return None
if keymap.TOGGLE_HELD in commands: if keymap.TOGGLE_HELD in commands:
if not len(self.listbox.body): if not len(self.listbox.body):
@ -461,6 +469,7 @@ class ChangeListView(urwid.WidgetWrap):
with self.app.db.getSession() as session: with self.app.db.getSession() as session:
change = session.getChange(change_key) change = session.getChange(change_key)
row.update(change, self.categories) row.update(change, self.categories)
self.advance()
return None return None
if keymap.TOGGLE_STARRED in commands: if keymap.TOGGLE_STARRED in commands:
if not len(self.listbox.body): if not len(self.listbox.body):
@ -472,6 +481,7 @@ class ChangeListView(urwid.WidgetWrap):
with self.app.db.getSession() as session: with self.app.db.getSession() as session:
change = session.getChange(change_key) change = session.getChange(change_key)
row.update(change, self.categories) row.update(change, self.categories)
self.advance()
return None return None
if keymap.TOGGLE_MARK in commands: if keymap.TOGGLE_MARK in commands:
if not len(self.listbox.body): if not len(self.listbox.body):
@ -483,9 +493,7 @@ class ChangeListView(urwid.WidgetWrap):
with self.app.db.getSession() as session: with self.app.db.getSession() as session:
change = session.getChange(change_key) change = session.getChange(change_key)
row.update(change, self.categories) row.update(change, self.categories)
if pos < len(self.listbox.body)-1: self.advance()
pos += 1
self.listbox.focus_position = pos
return None return None
if keymap.REFRESH in commands: if keymap.REFRESH in commands:
if self.project_key: if self.project_key: