Merge lp:~thisfred/u1db/cosas-delete into lp:u1db

Proposed by Eric Casteleijn on 2012-08-02
Status: Merged
Approved by: Eric Casteleijn on 2012-08-02
Approved revision: 370
Merged at revision: 370
Proposed branch: lp:~thisfred/u1db/cosas-delete
Merge into: lp:u1db
Diff against target: 44 lines (+19/-4)
1 file modified
cosas/ui.py (+19/-4)
To merge this branch: bzr merge lp:~thisfred/u1db/cosas-delete
Reviewer Review Type Date Requested Status
Roberto Alsina (community) 2012-08-02 Approve on 2012-08-02
Review via email: mp+117908@code.launchpad.net

Commit Message

- added keybindings for deleting and editing items (delete and enter, resp.)

Description of the Change

- added keybindings for deleting and editing items (delete and enter, resp.)

To post a comment you must log in.
Roberto Alsina (ralsina) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cosas/ui.py'
2--- cosas/ui.py 2012-08-01 20:58:39 +0000
3+++ cosas/ui.py 2012-08-02 14:40:29 +0000
4@@ -219,6 +219,23 @@
5 self.title_edit.clear()
6 # Give the edit field focus.
7 self.title_edit.setFocus()
8+ self.editing = False
9+
10+ def keyPressEvent(self, event):
11+ if event.key() == QtCore.Qt.Key_Delete:
12+ self.delete()
13+ return
14+ if event.key() == QtCore.Qt.Key_Return:
15+ if not self.editing:
16+ self.editing = True
17+ self.todo_list.openPersistentEditor(
18+ self.todo_list.currentItem())
19+ return
20+ else:
21+ self.todo_list.closePersistentEditor(
22+ self.todo_list.currentItem())
23+ self.editing = False
24+ super(Main, self).keyPressEvent(event)
25
26 def get_tag_color(self):
27 """Get a color number to use for a new tag."""
28@@ -281,14 +298,12 @@
29 def delete(self):
30 """Delete a todo item."""
31 # Delete the item from the database.
32- row = self.todo_list.currentRow()
33- item = self.todo_list.takeItem(row)
34+ index = self.todo_list.indexFromItem(self.todo_list.currentItem())
35+ item = self.todo_list.takeTopLevelItem(index.row())
36 if item is None:
37 return
38 self.store.delete_task(item.task)
39 # Clear the current selection.
40- self.todo_list.setCurrentRow(-1)
41- self.title_edit.clear()
42 self.item = None
43
44 def add_task(self, task):

Subscribers

People subscribed via source and target branches