Merge lp:~thisfred/u1db/colored-squares into lp:u1db

Proposed by Eric Casteleijn on 2012-08-13
Status: Merged
Approved by: Eric Casteleijn on 2012-08-14
Approved revision: 375
Merged at revision: 376
Proposed branch: lp:~thisfred/u1db/colored-squares
Merge into: lp:u1db
Diff against target: 74 lines (+13/-7)
2 files modified
cosas/cosas.ui (+6/-1)
cosas/ui.py (+7/-6)
To merge this branch: bzr merge lp:~thisfred/u1db/colored-squares
Reviewer Review Type Date Requested Status
Roberto Alsina (community) 2012-08-13 Approve on 2012-08-14
Review via email: mp+119376@code.launchpad.net

Commit Message

Used colored squares rather than row background colors for tag indicators.

Description of the Change

Used colored squares rather than row background colors for tag indicators.

To post a comment you must log in.
lp:~thisfred/u1db/colored-squares updated on 2012-08-13
375. By Eric Casteleijn on 2012-08-13

merged trunk

Roberto Alsina (ralsina) wrote :

I would really like this better if we drew a small round-cornered image instead of just using a full color square, but that's not code that belongs in an example application.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cosas/cosas.ui'
2--- cosas/cosas.ui 2012-08-10 12:08:33 +0000
3+++ cosas/cosas.ui 2012-08-13 16:11:19 +0000
4@@ -174,7 +174,7 @@
5 <bool>true</bool>
6 </property>
7 <property name="columnCount">
8- <number>2</number>
9+ <number>3</number>
10 </property>
11 <property name="topLevelItemCount" stdset="0">
12 <number>0</number>
13@@ -192,6 +192,11 @@
14 <string notr="true">2</string>
15 </property>
16 </column>
17+ <column>
18+ <property name="text">
19+ <string notr="true">3</string>
20+ </property>
21+ </column>
22 </widget>
23 </item>
24 </layout>
25
26=== modified file 'cosas/ui.py'
27--- cosas/ui.py 2012-08-13 14:28:46 +0000
28+++ cosas/ui.py 2012-08-13 16:11:19 +0000
29@@ -58,7 +58,7 @@
30 self._bg_color = color
31
32 def setData(self, column, role, value):
33- if column == 0:
34+ if column == 1:
35 if role == QtCore.Qt.CheckStateRole:
36 if value == QtCore.Qt.Checked:
37 self.task.done = True
38@@ -74,13 +74,13 @@
39 super(UITask, self).setData(column, role, value)
40
41 def data(self, column, role):
42- if role == QtCore.Qt.BackgroundRole:
43+ if role == QtCore.Qt.BackgroundRole and column == 0:
44 return self._bg_color
45 if role == QtCore.Qt.ForegroundRole:
46 if self.task.has_conflicts:
47 return CONFLICT_COLOR
48 return DONE if self.task.done else FOREGROUND
49- if column == 0:
50+ if column == 1:
51 if role == QtCore.Qt.FontRole:
52 font = self._font
53 font.setStrikeOut(self.task.done)
54@@ -93,7 +93,7 @@
55 return (
56 QtCore.Qt.Checked if self.task.done else
57 QtCore.Qt.Unchecked)
58- elif column == 1:
59+ elif column == 2:
60 if role == QtCore.Qt.DisplayRole:
61 return '!' if self.task.has_conflicts else ''
62 return super(UITask, self).data(column, role)
63@@ -245,8 +245,9 @@
64 self.store.initialize_db()
65 # hook up the delegate
66 header = self.todo_list.header()
67- header.setResizeMode(0, 1) # stretch first column
68- header.setResizeMode(1, 2) # second column fixed
69+ header.setResizeMode(0, 2) # first column fixed
70+ header.setResizeMode(1, 1) # stretch second column
71+ header.setResizeMode(2, 2) # third column fixed
72 header.setDefaultSectionSize(20)
73
74 header.setStretchLastSection(False)

Subscribers

People subscribed via source and target branches