Merge lp:~cjohnston/ubuntu-community-webthemes/960px into lp:ubuntu-community-webthemes/light-django-theme

Proposed by Chris Johnston
Status: Merged
Approved by: Michael Hall
Approved revision: 39
Merged at revision: 37
Proposed branch: lp:~cjohnston/ubuntu-community-webthemes/960px
Merge into: lp:ubuntu-community-webthemes/light-django-theme
Diff against target: 46 lines (+42/-0)
1 file modified
media/css/960.css (+42/-0)
To merge this branch: bzr merge lp:~cjohnston/ubuntu-community-webthemes/960px
Reviewer Review Type Date Requested Status
Michael Hall (community) Needs Fixing
Review via email: mp+68587@code.launchpad.net

Commit message

Adds 960px css file

To post a comment you must log in.
Revision history for this message
Tarmac WebDev (tarmac-webdev) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Revision history for this message
Michael Hall (mhall119) wrote :

Adding it to the base template will turn it on for all sites currently using this theme, and require that they specifically deactivate it. This is not the desired behavior. Instead let each site add that line to their own template when they want 960px.

review: Needs Fixing
39. By Chris Johnston

Removes from ubuntu_website_base

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'media/css/960.css'
2--- media/css/960.css 1970-01-01 00:00:00 +0000
3+++ media/css/960.css 2011-07-20 21:34:41 +0000
4@@ -0,0 +1,42 @@
5+/*
6+ * 960.css
7+ *
8+ * Style to allow the option of adding a 960px width to the
9+ * header and the footer of your Ubuntu Themed django app
10+ * while maintaining the ability to turn it off on certain
11+ * pages.
12+ *
13+ * To add the 960px to your app, add:
14+ * {% block 960css %}<link rel="stylesheet" type="text/css" href="{{ubuntu_website_media}}/css/960.css" />{% endblock %}
15+ * to your base.html
16+ *
17+ * To turn it off on a particular page, add:
18+ * {% block 960css %}{% endblock %}
19+ * to the page that you do not want to be 960px.
20+ *
21+ * Author: Chris Johnston ( cjohnston )
22+ *
23+ */
24+
25+aside#top-nav {
26+ width: 980px;
27+}
28+
29+header#page-header {
30+ width: 960px;
31+ margin: 0px auto;
32+ padding: 0px 10px;
33+ position: relative;
34+}
35+
36+footer#page-footer {
37+ width: 960px;
38+ margin: 0px auto;
39+ padding: 0px 10px;
40+ position: relative;
41+}
42+
43+#page-footer div.container {
44+ width: 940px;
45+}
46+

Subscribers

People subscribed via source and target branches