Ubuntu Apps Directory

Merge lp:~jonas-drange/ubuntu-webcatalog/always-include-carousel-css into lp:ubuntu-webcatalog

  1. always-include-carousel-css
  2. Merge into trunk
Proposed by Jonas G. Drange
Status: Merged
Approved by: Jonas G. Drange
Approved revision: 194
Merged at revision: 191
Proposed branch: lp:~jonas-drange/ubuntu-webcatalog/always-include-carousel-css
Merge into: lp:ubuntu-webcatalog
Diff against target: 28 lines (+6/-2)
2 files modified
src/webcatalog/templates/webcatalog/application_detail.html (+5/-0)
src/webcatalog/templates/webcatalog/index.html (+1/-2)
To merge this branch: bzr merge lp:~jonas-drange/ubuntu-webcatalog/always-include-carousel-css
Reviewer Review Type Date Requested Status
Stephen Stewart (community) Approve
Review via email: mp+185452@code.launchpad.net

Commit message

include the carousel css with application_detail template

Description of the change

include the carousel css with application_detail template

To post a comment you must log in.
lp:~jonas-drange/ubuntu-webcatalog/always-include-carousel-css updated
192. By Jonas G. Drange

moving out of base, into index and application_detail

Revision history for this message
Stephen Stewart (stephen-stewart) wrote :

I don't think you need the block.super if using css-extra, there should be nothing to override there

lp:~jonas-drange/ubuntu-webcatalog/always-include-carousel-css updated
193. By Jonas G. Drange

super is not strictly needed here

Revision history for this message
Stephen Stewart (stephen-stewart) :
review: Approve
lp:~jonas-drange/ubuntu-webcatalog/always-include-carousel-css updated
194. By Jonas G. Drange

removing css extra super in index

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/webcatalog/templates/webcatalog/application_detail.html'
2--- src/webcatalog/templates/webcatalog/application_detail.html 2013-09-03 12:52:17 +0000
3+++ src/webcatalog/templates/webcatalog/application_detail.html 2013-09-17 09:03:32 +0000
4@@ -9,6 +9,11 @@
5 {% endblock %}
6 {% block title %}{{ application.name }} — Ubuntu Apps Directory{% endblock %}
7 {% block header %}{{ application.name }}{% endblock %}
8+
9+{% block css-extra %}
10+<link rel="stylesheet" type="text/css" href="{% url 'wc-combo' %}?css/carousel.css"/>
11+{% endblock %}
12+
13 {% block js-base %}
14 {{ block.super }}
15 <script src="{% url 'wc-combo' %}?js/carousel.js&amp;js/screenshots.js"></script>
16
17=== modified file 'src/webcatalog/templates/webcatalog/index.html'
18--- src/webcatalog/templates/webcatalog/index.html 2013-09-03 12:52:17 +0000
19+++ src/webcatalog/templates/webcatalog/index.html 2013-09-17 09:03:32 +0000
20@@ -5,8 +5,7 @@
21 {% block header_container %}{% endblock %}
22 {% block search %}{% endblock %}
23
24-{% block css-base %}
25-{{ block.super }}
26+{% block css-extra %}
27 <link rel="stylesheet" type="text/css" href="{% url 'wc-combo' %}?css/carousel.css"/>
28 {% endblock %}
29

Subscribers

People subscribed via source and target branches