Merge lp:~svwilliams/summit/track-page into lp:~cjohnston/summit/track-page

Proposed by Stephen V. Williams
Status: Merged
Approved by: Chris Johnston
Approved revision: no longer in the source branch.
Merge reported by: Chris Johnston
Merged at revision: not available
Proposed branch: lp:~svwilliams/summit/track-page
Merge into: lp:~cjohnston/summit/track-page
Diff against target: 34 lines (+16/-4)
1 file modified
summit/schedule/templates/schedule/tracks.html (+16/-4)
To merge this branch: bzr merge lp:~svwilliams/summit/track-page
Reviewer Review Type Date Requested Status
Chris Johnston Pending
Review via email: mp+84350@code.launchpad.net

Commit message

tracks.html with div in for loop every 4th element and borrowed css from uds.ubuntu.com/tracks

Description of the change

The CSS is bad (needs a class, etc) However, this loops and every fourth element closes the list, closes the div and opens a new one. This follows the example at uds.ubuntu.com/tracks

To post a comment you must log in.
lp:~svwilliams/summit/track-page updated
232. By Chris Johnston

Fixes CSS provided by svwilliams

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'summit/schedule/templates/schedule/tracks.html'
2--- summit/schedule/templates/schedule/tracks.html 2011-12-02 02:30:47 +0000
3+++ summit/schedule/templates/schedule/tracks.html 2011-12-03 02:26:27 +0000
4@@ -6,14 +6,26 @@
5 <article class="main-content">
6 <h1>{{ summit.title }} Tracks</h1>
7
8-<ul class="tracks">
9 {% for track in summit.track_set.all %}
10-<li>
11+{% if forloop.counter == 1 %}
12+<div style="display:block;border-bottom-left-radius: 5px 5px;
13+border-bottom-right-radius: 5px 5px;clear: both;
14+padding: 5px;
15+margin-bottom: -3px;">
16+<ul class="tracks">
17+{% endif %}
18+<li>{{ forloop.counter }}
19 <h3><a href="/{{ summit.name }}/track/{{ track.slug|lower }}">{{ track.title }}</a></h3>
20 <p>{{ track.description }}</p>
21 </li>
22+{% if forloop.counter|divisibleby:"4" %}
23+</ul></div><div style="display:block;border-bottom-left-radius: 5px 5px;
24+border-bottom-right-radius: 5px 5px;clear: both;
25+padding: 5px;
26+margin-bottom: -3px;">
27+<ul class="tracks">
28+{% endif %}
29 {% endfor %}
30-</ul>
31-
32+</ul></div>
33 </article>
34 {% endblock %}

Subscribers

People subscribed via source and target branches