Merge lp:~elachuni/ubuntu-webcatalog/fix-404 into lp:ubuntu-webcatalog

Proposed by Anthony Lenton
Status: Merged
Approved by: Danny Tamez
Approved revision: 84
Merged at revision: 81
Proposed branch: lp:~elachuni/ubuntu-webcatalog/fix-404
Merge into: lp:ubuntu-webcatalog
Diff against target: 267 lines (+160/-80)
3 files modified
src/webcatalog/static/light/css/404.css (+90/-0)
src/webcatalog/templates/404.html (+70/-7)
src/webcatalog/templates/light/404.html (+0/-73)
To merge this branch: bzr merge lp:~elachuni/ubuntu-webcatalog/fix-404
Reviewer Review Type Date Requested Status
Danny Tamez (community) Approve
Review via email: mp+98237@code.launchpad.net

Commit message

Fixed the 404 error page so that it looks better, makes sense and is generally less navel-looking.

Description of the change

Details
=======
 - Added 404.css and a couple of images that were missing, from the light web-theme.
 - Fixed text per linked bug #585864
 - Made footer match the new one from webcatalog/base.html

To post a comment you must log in.
Revision history for this message
Danny Tamez (zematynnad) wrote :

could get rid of line 7 and line 8 only needs to be
margin: 0;

Why were the changes made to light/404.html (as we're no longer extending it)? Is it for others that might use it?

83. By Anthony Lenton

Removed unused 404.html.

84. By Anthony Lenton

Css fix per code review.

Revision history for this message
Danny Tamez (zematynnad) wrote :

Looks great.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'src/webcatalog/static/light/css/404.css'
2--- src/webcatalog/static/light/css/404.css 1970-01-01 00:00:00 +0000
3+++ src/webcatalog/static/light/css/404.css 2012-03-19 19:45:22 +0000
4@@ -0,0 +1,90 @@
5+/* 404 */
6+#template-err2 #row-main-row .pane-content{
7+ margin: 0;
8+ background:url('/assets/light/images/header_error_large_ubuntu.png') no-repeat;
9+ padding-left: 8px;
10+ width: 944px;
11+ height: 244px;
12+}
13+
14+#template-err2 #row-main-row .pane-content .img_404 {
15+ width: 207px;
16+ height: 247px;
17+ float: left;
18+}
19+
20+#template-err2 #row-main-row .pane-content .img_404 h1{
21+ color: #fff;
22+ margin: 85px 30px;
23+}
24+
25+#template-err2 #row-main-row .pane-content h2 {
26+ padding-bottom:8px;
27+ padding-top:24px;
28+}
29+
30+#template-err2 #row-main-row .pane-content h3 {
31+ padding-bottom:8px;
32+ padding-top:8px;
33+}
34+
35+#template-err2 #row-main-row .pane-content p {
36+ color: #AEA79F;
37+ padding-bottom:8px;
38+}
39+
40+#template-err2 #row-2 .panel-pane {
41+ min-height: 192px;
42+ padding-bottom:32px;
43+ margin-bottom:72px;
44+ background-color:#f7f7f7;
45+}
46+
47+#template-err2 #row-2 h2 {
48+ font-weight: normal;
49+ margin: 8px;
50+ width: 210px;
51+ color: #333333;
52+ letter-spacing: -1px;
53+}
54+
55+#template-err2 #row-2 h2 a {
56+ background: url("/assets/light/images/darrow.png") no-repeat right 8px;
57+ padding: 1px 13px 0px 0px;
58+ display: inline;
59+}
60+
61+#template-err2 #row-2 p {
62+ padding-left: 8px;
63+ width: 130px;
64+}
65+
66+.row-wrapper-2 {
67+ margin-left: 8px;
68+}
69+
70+.grid-4 {
71+ background:#f7f7f7;
72+ width: 304px;
73+ -moz-border-radius: 4px;
74+ -webkit-border-radius: 4px;
75+ display: inline;
76+ float: left;
77+ margin-left: 8px;
78+ margin-right: 8px;
79+ min-height: 192px;
80+ padding-bottom: 32px;
81+ margin-bottom: 72px;
82+ position: relative;
83+}
84+
85+div.err2_pane_4 img {
86+ position: absolute;
87+ left: 140px;
88+ top: -60px;
89+ clip: rect(131px 159px 268px 63px);
90+}
91+.clear {
92+ clear: both;
93+}
94+
95
96=== added file 'src/webcatalog/static/light/images/darrow.png'
97Binary files src/webcatalog/static/light/images/darrow.png 1970-01-01 00:00:00 +0000 and src/webcatalog/static/light/images/darrow.png 2012-03-19 19:45:22 +0000 differ
98=== added file 'src/webcatalog/static/light/images/header_error_large_ubuntu.png'
99Binary files src/webcatalog/static/light/images/header_error_large_ubuntu.png 1970-01-01 00:00:00 +0000 and src/webcatalog/static/light/images/header_error_large_ubuntu.png 2012-03-19 19:45:22 +0000 differ
100=== modified file 'src/webcatalog/templates/404.html'
101--- src/webcatalog/templates/404.html 2011-07-04 11:52:26 +0000
102+++ src/webcatalog/templates/404.html 2012-03-19 19:45:22 +0000
103@@ -1,4 +1,4 @@
104-{% extends "light/404.html" %}
105+{% extends "webcatalog/base.html" %}
106 {% load i18n %}
107
108 {% comment %}
109@@ -6,11 +6,74 @@
110 GNU Affero General Public License version 3 (see the file LICENSE).
111 {% endcomment %}
112
113-{% block content_container_extra %}
114- <div>
115- <p>
116- {% blocktrans %}(Error <abbr>ID</abbr>: {{ oopsid }}){% endblocktrans %}
117- </p>
118- </div>
119+{% block title %}Error | Ubuntu{% endblock %}
120+
121+{% block header %}{% endblock %}
122+
123+{% block head_extra %}
124+ <link rel="stylesheet" type="text/css" href="{% url wc-combo %}?light/css/reset.css&light/css/styles.css&light/css/404.css&light/css/forms.css"/>
125+{% endblock %}
126+
127+{% block content_container %}
128+<div id="template-err2">
129+ <div id="row-main-row" >
130+ <div class="pane-content">
131+ <div class="img_404">
132+ <h1>404</h1>
133+ </div>
134+
135+ <h2>Page not found.</h2>
136+ <h3>Sorry, we couldn't find the page you were looking for</h3>
137+ {% block content_container_default %}
138+ <p>Please use the search box above to find what you need.
139+ Alternatively, you can make a fresh start at the
140+ <a href="http://www.ubuntu.com">Ubuntu home page</a>.
141+ </p>
142+ {% endblock %}
143+ </div>
144+ </div>
145+ <div>
146+ <p>
147+ {% blocktrans %}(Error <abbr>ID</abbr>: {{ oopsid }}){% endblocktrans %}
148+ </p>
149+ </div>
150+
151+ <div class="clear"> </div>
152+
153+ <div id="row-2" class="row-wrapper-2">
154+
155+ <div class="grid-4 err2_pane_4" >
156+ <div>
157+ <h2><a href="http://www.ubuntu.com/project">About Ubuntu</a></h2>
158+ <p></p>
159+ <p>Find out more about the Ubuntu Project and how you can contribute.</p>
160+ <p></p>
161+ <p><img src="http://www.ubuntu.com/sites/default/files/active/error_page_imagery/pictogram_knowledge_medium.png" alt="pictogram_knowledge_medium" /></p>
162+ </div>
163+ </div>
164+
165+ <div class="grid-4 err2_pane_4" >
166+ <div>
167+ <h2><a href="http://www.ubuntu.com/desktop/get-ubuntu">Get Ubuntu</a></h2>
168+ <p></p>
169+ <p>Download Ubuntu today.</p>
170+ <p></p>
171+ <p><img src="http://www.ubuntu.com/sites/default/files/active/03_global/02_pictograms/pictogram_download_medium.png" alt="pictogram_download_medium" /></p>
172+ </div>
173+ </div>
174+
175+ <div class="grid-4 err2_pane_4" >
176+ <div>
177+ <h2><a href="http://shop.canonical.com">Visit the shop</a></h2>
178+ <p></p>
179+ <p>Buy Ubuntu CDs, accessories or training at the Canonical shop.</p>
180+ <p></p>
181+ <p><img src="http://www.ubuntu.com/sites/default/files/active/error_page_imagery/pictogram_shop_medium.png" alt="pictogram_shop_medium.png" /></p>
182+ </div>
183+ </div>
184+
185+ <div class="clear"> </div>
186+ </div>
187+</div>
188 {% endblock %}
189
190
191=== removed file 'src/webcatalog/templates/light/404.html'
192--- src/webcatalog/templates/light/404.html 2012-03-05 17:41:06 +0000
193+++ src/webcatalog/templates/light/404.html 1970-01-01 00:00:00 +0000
194@@ -1,73 +0,0 @@
195-{% extends 'light/index.1col.html' %}
196-
197-{% block title %}Error | Ubuntu{% endblock %}
198-
199-{% block header %}{% endblock %}
200-
201-{% block head_extra %}
202- <link rel="stylesheet" type="text/css" href="{% url wc-combo %}?light/css/reset.css&light/css/styles.css&light/css/404.css&light/css/forms.css"/>
203-{% endblock %}
204-
205-{% block content_container %}
206-<div id="template-err2">
207- <div id="row-main-row" >
208- <div class="pane-content">
209- <div class="img_404">
210- <h1>404</h1>
211- </div>
212- <h2>This page does not exist.</h2>
213- <h3>Well, obviously <em>this</em> page exists. But the
214- page you requested does not exist. This page is just
215- here to tell you that the page you requested does not exist.
216- </h3>
217- {% block content_container_default %}
218- <p>You can use the search box above to find what you need.
219- Or you can make a fresh start at the
220- <a href="http://www.ubuntu.com">Ubuntu home page</a>.
221- </p>
222- <p>Still can't find what you're looking for?
223- <a href="/contact-us">Get in touch today</a>.
224- </p>
225- {% endblock %}
226- </div>
227- </div>
228- {% block content_container_extra %}{% endblock %}
229-
230- <div class="clear"> </div>
231-
232- <div id="row-2" class="row-wrapper-2">
233-
234- <div class="grid-4 err2_pane_4" >
235- <div>
236- <h2><a href="http://www.ubuntu.com/project">About Ubuntu</a></h2>
237- <p></p>
238- <p>Find out more about the Ubuntu Project and how you can contribute.</p>
239- <p></p>
240- <p><img src="http://www.ubuntu.com/sites/default/files/active/error_page_imagery/pictogram_knowledge_medium.png" alt="pictogram_knowledge_medium" /></p>
241- </div>
242- </div>
243-
244- <div class="grid-4 err2_pane_4" >
245- <div>
246- <h2><a href="http://www.ubuntu.com/desktop/get-ubuntu">Get Ubuntu</a></h2>
247- <p></p>
248- <p>Download Ubuntu today.</p>
249- <p></p>
250- <p><img src="http://www.ubuntu.com/sites/default/files/active/03_global/02_pictograms/pictogram_download_medium.png" alt="pictogram_download_medium" /></p>
251- </div>
252- </div>
253-
254- <div class="grid-4 err2_pane_4" >
255- <div>
256- <h2><a href="http://shop.canonical.com">Visit the shop</a></h2>
257- <p></p>
258- <p>Buy Ubuntu CDs, accessories or training at the Canonical shop.</p>
259- <p></p>
260- <p><img src="http://www.ubuntu.com/sites/default/files/active/error_page_imagery/pictogram_shop_medium.png" alt="pictogram_shop_medium.png" /></p>
261- </div>
262- </div>
263-
264- <div class="clear"> </div>
265- </div>
266-</div>
267-{% endblock %}

Subscribers

People subscribed via source and target branches