ttygroup/boartty
Code Issues Proposed changes

Allow reviewing one change in change list

Browse Source 
Reviewing multiple changes is accomplished by setting the process
mark; also allow reviewing a single change from the change list
if the process mark is not set.

Change-Id: Id615ab084d576307fd2b13d31a20842763c9aca7
This commit is contained in:
James E. Blair 2015-05-23 15:40:01 -07:00
parent 2b302321ba
commit 9cdfa07f91
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
gertty/view/change_list.py
View File

@ -497,9 +497,11 @@ class ChangeListView(urwid.WidgetWrap):
self.app.status.update() self.app.status.update()
return None return None
if keymap.REVIEW in commands: if keymap.REVIEW in commands:
marked = [row for row in self.change_rows.values() if row.mark] rows = [row for row in self.change_rows.values() if row.mark]
if marked: if not rows:
self.openReview(marked) pos = self.listbox.focus_position
rows = [self.listbox.body[pos]]
self.openReview(rows)
return None return None
if keymap.SORT_BY_NUMBER in commands: if keymap.SORT_BY_NUMBER in commands:
if not len(self.listbox.body): if not len(self.listbox.body):