Merge lp:~jkakar/kanban/suppress-uncategorized-lane into lp:kanban

Proposed by Jamu Kakar
Status: Merged
Approved by: Martin Pool
Approved revision: 25
Merged at revision: 25
Proposed branch: lp:~jkakar/kanban/suppress-uncategorized-lane
Merge into: lp:kanban
Diff against target: 49 lines (+14/-6)
2 files modified
kanban/templates/kanban.html (+9/-5)
media/kanban.css (+5/-1)
To merge this branch: bzr merge lp:~jkakar/kanban/suppress-uncategorized-lane
Reviewer Review Type Date Requested Status
Martin Pool Approve
Review via email: mp+53151@code.launchpad.net

Description of the change

This branch introduces the following changes:

- The 'uncategorized' story name is only shown when there is more than
  one story displayed on the board.

To post a comment you must log in.
Revision history for this message
Martin Pool (mbp) :
review: Approve
Revision history for this message
Martin Pool (mbp) wrote :

This has some conflicts; i'll resolve them.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'kanban/templates/kanban.html'
2--- kanban/templates/kanban.html 2011-02-17 10:07:00 +0000
3+++ kanban/templates/kanban.html 2011-03-12 22:12:02 +0000
4@@ -27,13 +27,17 @@
5 </div>
6
7 {% for story in kanban_board.stories %}
8- <div class="tiles row story">
9- {% if story.name %}
10- <div class="position-0 width-12 cell">{{ story.name }}</div>
11+ {% if kanban_board.stories|length > 1 %}
12+ <div class="tiles row story">
13+ {% if story.name %}
14+ <div class="position-0 width-12 cell">{{ story.name }}</div>
15+ {% else %}
16+ <div class="position-0 width-12 cell">uncategorized</div>
17+ {% endif %}
18+ </div>
19 {% else %}
20- <div class="position-0 width-12 cell">uncategorized</div>
21+ <div class="tiles row no-story"></div>
22 {% endif %}
23- </div>
24
25 <div class="tiles row">
26 <div class="position-0 width-2 cell">
27
28=== modified file 'media/kanban.css'
29--- media/kanban.css 2011-02-08 15:16:14 +0000
30+++ media/kanban.css 2011-03-12 22:12:02 +0000
31@@ -95,13 +95,17 @@
32 padding-bottom: 12px;
33 }
34
35-.story {
36+.story, .no-story {
37 color: black;
38 font-weight: bolder;
39 border-top: 1px dotted #dddddd;
40 padding-top: 12px;
41 }
42
43+.no-story {
44+ padding-top: 0;
45+}
46+
47 #prefooter {
48 line-height: 0;
49 -moz-border-radius-bottomleft: 6px;

Subscribers

People subscribed via source and target branches

to all changes: