Merge lp:~brett-alton-deactivatedaccount/quickshot/website into lp:~quickshotdevs/quickshot/website

Proposed by Brett Alton
Status: Merged
Approved by: Brett Alton
Approved revision: 6
Merge reported by: Adnane Belmadiaf
Merged at revision: not available
Proposed branch: lp:~brett-alton-deactivatedaccount/quickshot/website
Merge into: lp:~quickshotdevs/quickshot/website
Diff against target: 266 lines (+106/-39)
6 files modified
media/css/styles.css (+24/-5)
sitemap.py (+1/-1)
templates/base.html (+5/-5)
templates/contact.html (+46/-22)
templates/install.html (+25/-1)
views.py (+5/-5)
To merge this branch: bzr merge lp:~brett-alton-deactivatedaccount/quickshot/website
Reviewer Review Type Date Requested Status
Brett Alton (community) Approve
Review via email: mp+34648@code.launchpad.net

Description of the change

Accidentally put under my own branch. Please merge and I will develop under the proper branch from now on.

To post a comment you must log in.
Revision history for this message
Brett Alton (brett-alton-deactivatedaccount) :
review: Approve
Revision history for this message
Adnane Belmadiaf (daker) wrote :

Merged!!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'media/css/styles.css'
2--- media/css/styles.css 2010-08-27 23:28:06 +0000
3+++ media/css/styles.css 2010-09-05 23:48:43 +0000
4@@ -58,6 +58,7 @@
5 font-family: Bitstream Vera Sans Mono, Dejavu sans mono, monospace;
6 display: block;
7 padding: 0.5em 0 0.5em 2em;
8+ font-size: 85%;
9 }
10
11 #page {
12@@ -115,7 +116,8 @@
13 float:left;
14 }
15
16-#page-content .content {
17+#page-content .content,
18+#right .content {
19 line-height:20px;
20 margin-bottom:20px;
21 font-size:14px;
22@@ -123,7 +125,6 @@
23
24 #picture {
25 float:right;
26- width: 350px;
27 height:300px;
28 }
29
30@@ -223,6 +224,12 @@
31 min-width:250px; height:30px;
32 }
33
34+.medium.button, .medium.button:visited {
35+ font-size: 22px;
36+ padding: 12px 10px 10px;
37+ min-width:150px; height:25px;
38+}
39+
40 .orange.button, .orange.button:visited { background-color: #FB8B00; }
41 .orange.button:focus { background-color: #F44800; }
42
43@@ -261,13 +268,12 @@
44 }
45
46 #contactform {
47- width: 300px;
48- margin-top: 25px;
49+ width: 400px;
50 }
51
52 #contactform input, #contactform textarea {
53 padding: 5px;
54- width: 200px;
55+ width: 300px;
56 font-size: 1.4em;
57 margin: 0px 0px 5px 0px;
58 border: 1px solid #ccc;
59@@ -284,6 +290,7 @@
60 #contactform input.submit-button {
61 width: 100px;
62 float: right;
63+ margin-right: 23px;
64 }
65
66 label {
67@@ -295,3 +302,15 @@
68 font-size: 1.4em;
69 }
70
71+span.left{
72+ font-weight: bold;
73+ float: left;
74+ margin-right: 10px;
75+ text-align: left;
76+ width: 75px;
77+}
78+
79+span.right{
80+ float: left;
81+ text-align: left;
82+}
83
84=== modified file 'sitemap.py'
85--- sitemap.py 2010-07-09 21:53:51 +0000
86+++ sitemap.py 2010-09-05 23:48:43 +0000
87@@ -6,7 +6,7 @@
88
89 def get_urls(self, page=1):
90 urls = []
91- pages = ('', 'install', 'getinvolved', 'contact', 'download',)
92+ pages = ('', 'install', 'contact', 'download',)
93 for page in pages:
94 sep = ''
95 if page:
96
97=== modified file 'templates/base.html'
98--- templates/base.html 2010-07-14 12:08:59 +0000
99+++ templates/base.html 2010-09-05 23:48:43 +0000
100@@ -38,11 +38,11 @@
101 <div id="footer_links">
102 <span class="copyright">&copy; 2009 - {% now "Y" %} Ubuntu Manual Team, some rights reserved</span><br />
103 <ul class="links">
104- <li><a href="#">Quickshot</a></li>
105- <li><a href="#">USLC</a></li>
106- <li><a href="#">Our philosophy</a></li>
107- <li><a href="#">Report a bug</a></li>
108- <li><a href="#">Contact us</a></li>
109+ <li><a href="{% url home %}" title="Quickshot">Quickshot</a></li>
110+ <!--<li><a href="#">USLC</a></li>-->
111+ <!--<li><a href="#">Our philosophy</a></li>-->
112+ <li><a href="https://bugs.launchpad.net/quickshot/+filebug">Report a bug</a></li>
113+ <li><a href="{% url contact %}" title="contact">Contact us</a></li>
114 </ul>
115 </div>
116 </div>
117
118=== modified file 'templates/contact.html'
119--- templates/contact.html 2010-08-27 23:28:06 +0000
120+++ templates/contact.html 2010-09-05 23:48:43 +0000
121@@ -5,29 +5,53 @@
122
123 {% block content %}
124 <div class="entry-content">
125-
126- {% if status %}
127- {% if status.good %}
128- <p>Your email has been sent. Ben will get back to you as soon as he possibly can!</p>
129+ <div id="page-content">
130+ {% if status %}
131+ {% if status.good %}
132+ <p>Your email has been sent. The Quickshot developers will get back to you as soon as they possibly can!</p>
133+ {% else %}
134+ <p>Oops, error: {{ status.msg }}</p>
135+ {% endif %}
136 {% else %}
137- <p>Oops, error: {{ status.msg }}</p>
138- {% endif %}
139- {% else %}
140- {{ page.gethtml|safe }}
141- <div id="contactform">
142- <form id="contact" action="/contact" method="POST">
143- <label for="id_sender">Email:</label>
144-
145- <label for="City">Subject:</label>
146- <input type="text" name="City" id="City" />
147-
148- <label for="Message">Comments:</label><br />
149- <textarea name="Message" rows="20" cols="20" id="Message"></textarea>
150-
151- <input type="submit" name="submit" value="Submit" class="submit-button" />
152- </form>
153- </div>
154-
155+ {{ page.gethtml|safe }}
156+ <div id="contactform">
157+ <form id="contact" action="/contact" method="POST">
158+ <div class="required">
159+ <label for="name">Name:</label>
160+ <input type="text" name="name" id="name" class="inputText" size="10" maxlength="100" value="" />
161+ </div>
162+ <div class="required">
163+ <label for="email">E-mail:</label>
164+ <input type="text" name="email" id="email" class="inputText" size="10" maxlength="100" value="" />
165+ </div>
166+ <div class="required">
167+ <label for="message">Message:</label><br />
168+ <textarea name="message" rows="20" cols="20" id="message"></textarea>
169+ </div>
170+ <div class="required">
171+ <input type="submit" name="submit" value="Submit" class="submit-button" />
172+ </div>
173+ </form>
174+ </div>
175+ </div>
176+
177+ <div id="right" style="float: left;">
178+ <div class="content">
179+ <span class="left">IRC:</span><span class="right">#ubuntu-manual on <em>irc.freenode.net</em></span>
180+ </div>
181+ <div class="content">
182+ <span class="left">Email:</span><span class="right">quickshot@list.launchpad.net</span>
183+ </div>
184+ <div class="content">
185+ <span class="left">Launchpad:</span><span class="right"><a href="https://launchpad.net/~quickshotdevs">https://launchpad.net/~quickshotdevs</a></span>
186+ </div>
187+ <div class="content">
188+ <span class="left">Facebook:</span><span class="right"><a href="http://facebook.com/quickshot">http://facebook.com/quickshot</a></span>
189+ </div>
190+ <div class="content">
191+ <span class="left">Twitter:</span><span class="right"><a href="http://twitter.com/quickshotteam">http://twitter.com/quickshotteam</a></span>
192+ </div>
193+ </div>
194
195 {% endif %}
196
197
198=== modified file 'templates/install.html'
199--- templates/install.html 2010-07-09 21:53:51 +0000
200+++ templates/install.html 2010-09-05 23:48:43 +0000
201@@ -4,5 +4,29 @@
202 {% block section %}download{% endblock %}
203
204 {% block content %}
205-
206+<div id="page-content">
207+
208+ <div class="content">
209+ If you are not already using Ubuntu 10.10 (Maverick Meerkat), the easiest, and recommended way to use Quickshot is to download the Quickshot Live CD and boot into that to take screenshots.
210+ </div>
211+
212+
213+ <div class="content">
214+ <a href="http://files.ubuntu-manual.org/quickshot/ubuntu-manual-quickshot-i386-0.0.8.iso" class="medium button orange" style="float: left; margin-right: 8px">Download</a>
215+ <span style="font-size: 85%"><strong>Please note:</strong> This Live CD is a striped down version of Ubuntu 10.10 and therefore is not suitable for use as a real operating system.</span>
216+ </div>
217+
218+ <div class="content">
219+ Already using Ubuntu 10.10? Add our PPA by running these commands:
220+ <code>
221+ sudo add-apt-repository ppa:quickshotdevs/quickshot-release<br />
222+ sudo apt-get update<br />
223+ sudo apt-get install quickshot
224+ </code>
225+ </div>
226+</div>
227+
228+<div id="picture">
229+ <img src="/static/images/quickshot.png" >
230+</div>
231 {% endblock %}
232
233=== modified file 'views.py'
234--- views.py 2010-07-09 21:53:51 +0000
235+++ views.py 2010-09-05 23:48:43 +0000
236@@ -15,9 +15,9 @@
237
238
239 class ContactForm(forms.Form):
240- subject = forms.CharField(max_length=100)
241+ name = forms.CharField(max_length=100)
242+ email = forms.EmailField()
243 message = forms.CharField(widget=forms.widgets.Textarea())
244- sender = forms.EmailField()
245
246 def contactpost(req):
247 """
248@@ -29,15 +29,15 @@
249 form = ContactForm(req.POST) # A form bound to the POST data
250
251 if form.is_valid(): # All validation rules pass
252- subject = form.cleaned_data['subject']
253+ name = form.cleaned_data['name']
254+ email = form.cleaned_data['email']
255 message = form.cleaned_data['message']
256- sender = form.cleaned_data['sender']
257 recipients = [deploy.EMAIL_CONTACT_RECIPIENT]
258
259 from django.core.mail import EmailMessage
260
261 try:
262- em = EmailMessage(subject, message, deploy.EMAIL_CONTACT_SENDER, recipients, headers = {'Reply-To': sender})
263+ em = EmailMessage(name, message, deploy.EMAIL_CONTACT_SENDER, recipients, headers = {'Reply-To': email})
264 em.send()
265 status = { 'good': True, 'msg':"" }
266 except SMTPException, ex:

Subscribers

People subscribed via source and target branches