From 7938aa91d2889536af6a2f3857ae0a2dc7cf0a42 Mon Sep 17 00:00:00 2001
From: Prem Sichanugrist <psichanugrist@thoughtbot.com>
Date: Mon, 30 Apr 2012 12:01:31 -0400
Subject: [PATCH] Use inline if-else to cleanup HTML

The result from the normal if-else block has linebreak, which result in
poorly formatted HTML code. This should fix it and make the code looks
cleaner.
---
 templates/accordion.html | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/templates/accordion.html b/templates/accordion.html
index a94f738d29..9a8f7ddefe 100644
--- a/templates/accordion.html
+++ b/templates/accordion.html
@@ -7,21 +7,12 @@
 
     <ul>
       % for section in chapter['sections']:
-      <li
-          % if 'active' in section and section['active']:
-            class="active"
-          % endif
-        >
-
+        <li${' class="active"' if 'active' in section and section['active'] else ''}>
           <a href="${reverse('courseware_section', args=format_url_params([course_name, chapter['name'], section['name']]))}">
             <p>${section['name']}</p>
 
             <p class="subtitle">
-              ${section['format']}
-
-              % if 'due' in section and section['due']!="":
-                due ${section['due']}
-              % endif
+              ${section['format']} ${"due " + section['due'] if 'due' in section and section['due'] != '' else ''}
             </p>
           </a>
         % endfor