harvest-dev team mailing list archive

[Daniel Holbach <daniel.holbach@xxxxxxxxxx>]

You have been requested to review the proposed merge of lp:~dholbach/harvest/bug518887 into lp:harvest.



-- 
https://code.launchpad.net/~dholbach/harvest/bug518887/+merge/21548
Your team harvest-dev is requested to review the proposed merge of lp:~dholbach/harvest/bug518887 into lp:harvest.

=== modified file 'harvest/opportunities/templates/opportunities/opportunities_by_type.html'
--- harvest/opportunities/templates/opportunities/opportunities_by_type.html	2010-02-22 15:43:51 +0000
+++ harvest/opportunities/templates/opportunities/opportunities_by_type.html	2010-03-17 11:54:18 +0000
@@ -13,55 +13,16 @@
   {% for type in types.object_list %}
     <li>
       <div class="package_name threshold{{ type.opportunity_class }}">
-        <span class="name">{{ type.name }}</span>
+        <span class="name"><a href="{{ type.get_absolute_url }}">{{ type.name }}</a></span>
         <span class="count">
             ({{ type.opportunity_set.count }} issues)
         </span>
       </div>
-      <div class="package_details">
-        <ul>
-        {% for opportunity in type.opportunity_set.all %}
-            {% include "opportunities/opportunity_detail.inc.html" %}
-        {% endfor %}
-        </ul>
-      </div>
-    </li>
   {% endfor %}
   </ul>
-
-  <div class="pagination">
-    <span class="step-links">
-    {% if types.has_previous %}
-      <a href="?page={{ types.previous_page_number }}">previous</a>
-    {% endif %}
-
-    <span class="current">
-      Page {{ types.number }} of {{ types.paginator.num_pages }}.
-    </span>
-
-    {% if types.has_next %}
-      <a href="?page={{ types.next_page_number }}">next</a>
-    {% endif %}
-    </span>
-  </div>
-
 {% else %}
   <p>{% trans "There are currently no opportunities in Harvest. :(" %}</p>
 {% endif %}
 
 </div>
-
-<script type="text/javascript">
-YUI().use('node', function(Y) {
-
-  Y.all('.package_name').on('click', function(e) {
-    // XXX: rockstar: Currently, this is kinda jumpy.  It'd be nice if it was
-    // all sexy animated.
-    Y.all('.package_details').setStyle('display', 'none');
-    e.currentTarget.next().setStyle('display', 'block');
-  });
-
-});
-
-</script>
 {% endblock %}