Merge lp:~mwhudson/lava-dispatcher/style into lp:lava-dispatcher

Proposed by Michael Hudson-Doyle
Status: Rejected
Rejected by: Michael Hudson-Doyle
Proposed branch: lp:~mwhudson/lava-dispatcher/style
Merge into: lp:lava-dispatcher
Diff against target: 129 lines (+52/-17)
5 files modified
lava/scheduler/interface/static/css/style.css (+12/-0)
lava/scheduler/interface/static/js/scheduler_ajax.js (+3/-3)
lava/scheduler/interface/templates/base.html (+29/-11)
lava/scheduler/interface/templates/scheduler/index.html (+7/-1)
lava/scheduler/interface/urls.py (+1/-2)
To merge this branch: bzr merge lp:~mwhudson/lava-dispatcher/style
Reviewer Review Type Date Requested Status
Paul Larson Pending
Review via email: mp+57259@code.launchpad.net

Description of the change

This branch adds a style sheet, moves the only page to the front of the application, tidies up the base template a bit and adds a terrible, terrible logo. You can see what the page looks like with these changes at http://people.linaro.org/~mwh/index.png.

To post a comment you must log in.
Revision history for this message
Paul Larson (pwlars) wrote :

Wow, that really is a bad logo! :)
I had actually thought the launch-control red theme was kind of fitting for lava as well, and it would also be nice for consistency across the tools. But possibly with some simple logo, yes.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Let's bash mirsad's branch into something usable instead.

Unmerged revisions

32. By Michael Hudson-Doyle

add hilariously terrible logo
add favicon
add stylesheet
fix template inheritance
make base template a bit nicer

31. By Michael Hudson-Doyle

fix lint in js

30. By Michael Hudson-Doyle

move job submission to site root

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== added directory 'lava/scheduler/interface/static/css'
=== added file 'lava/scheduler/interface/static/css/style.css'
--- lava/scheduler/interface/static/css/style.css 1970-01-01 00:00:00 +0000
+++ lava/scheduler/interface/static/css/style.css 2011-04-11 23:20:52 +0000
@@ -0,0 +1,12 @@
1body {
2 font-family: sans;
3}
4a img { border: none; }
5div.content {
6 margin-left: auto;
7 margin-right: auto;
8 width: 75%;
9}
10img.logo {
11 float: left;
12}
013
=== added directory 'lava/scheduler/interface/static/images'
=== added file 'lava/scheduler/interface/static/images/linaro.ico'
1Binary files lava/scheduler/interface/static/images/linaro.ico 1970-01-01 00:00:00 +0000 and lava/scheduler/interface/static/images/linaro.ico 2011-04-11 23:20:52 +0000 differ14Binary files lava/scheduler/interface/static/images/linaro.ico 1970-01-01 00:00:00 +0000 and lava/scheduler/interface/static/images/linaro.ico 2011-04-11 23:20:52 +0000 differ
=== added file 'lava/scheduler/interface/static/images/logo.png'
2Binary files lava/scheduler/interface/static/images/logo.png 1970-01-01 00:00:00 +0000 and lava/scheduler/interface/static/images/logo.png 2011-04-11 23:20:52 +0000 differ15Binary files lava/scheduler/interface/static/images/logo.png 1970-01-01 00:00:00 +0000 and lava/scheduler/interface/static/images/logo.png 2011-04-11 23:20:52 +0000 differ
=== modified file 'lava/scheduler/interface/static/js/scheduler_ajax.js'
--- lava/scheduler/interface/static/js/scheduler_ajax.js 2011-03-21 17:49:27 +0000
+++ lava/scheduler/interface/static/js/scheduler_ajax.js 2011-04-11 23:20:52 +0000
@@ -1,11 +1,11 @@
1function get_test_cases(){1function get_test_cases(){
2 new Ajax.Request('/scheduler/', { 2 new Ajax.Request('/', {
3 method: 'post',3 method: 'post',
4 parameters: $H({'test_suite':$('id_test_suite').getValue()}),4 parameters: $H({'test_suite':$('id_test_suite').getValue()}),
5 onSuccess: function(transport) {5 onSuccess: function(transport) {
6 var e = $('id_test_case')6 var e = $('id_test_case');
7 if(transport.responseText)7 if(transport.responseText)
8 e.update(transport.responseText)8 e.update(transport.responseText);
9 }9 }
10 }); // end new Ajax.Request10 }); // end new Ajax.Request
11}11}
1212
=== modified file 'lava/scheduler/interface/templates/base.html'
--- lava/scheduler/interface/templates/base.html 2011-04-07 04:01:20 +0000
+++ lava/scheduler/interface/templates/base.html 2011-04-11 23:20:52 +0000
@@ -1,16 +1,34 @@
1<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">1<?xml version="1.0" encoding="utf-8"?>
2<html lang="en">2<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
3 "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
4<html xmlns="http://www.w3.org/1999/xhtml" lang="en" xml:lang="en">
3 <head>5 <head>
4 <title>Lava: Submit test job</title>6 <script type="text/javascript" src="{{MEDIA_URL}}js/prototype.js"></script>
5 <script type="text/javascript" src="/scheduler/static/js/prototype.js"></script>7 <script type="text/javascript" src="{{MEDIA_URL}}js/scheduler_ajax.js"></script>
6 <script type="text/javascript" src="/scheduler/static/js/scheduler_ajax.js"></script>8 <link rel="shortcut icon" href="{{MEDIA_URL}}images/linaro.ico" />
9 <link rel="stylesheet" type="text/css" href="{{MEDIA_URL}}css/style.css" />
10 <title>{% block title %}LAVA{% endblock %}</title>
7 </head>11 </head>
8 <body>12 <body>
9 {% if user.is_authenticated %}13 <div class="content">
10 Logged in as {{ user.username }} - <a href="{{ logout_url }}">log out</a>14 <p id="login-p" style="text-align: right; margin: 0 0 0 0; padding: 0 0 0 0">
11 {% else %}15 {% if user.is_authenticated %}
12 <a id="log-in" href="{{ login_url }}">Login</a>16 Logged in as {{ user.username }} - <a href="{{ logout_url }}">log out</a>
13 {% endif %}17 {% else %}
14 {% block content %}{% endblock %}18 <a id="log-in" href="{{ login_url }}">Login</a>
19 {% endif %}
20 </p>
21 <table>
22 <tr>
23 <td>
24 <a href="{{ root }}"><img src="{{MEDIA_URL}}images/logo.png" class="logo" /></a>
25 </td>
26 <td>
27 <h1>{% block heading %}LAVA{% endblock %}</h1>
28 </td>
29 </tr>
30 </table>
31 {% block content %}{% endblock %}
32 </div>
15 </body>33 </body>
16</html>34</html>
1735
=== modified file 'lava/scheduler/interface/templates/scheduler/index.html'
--- lava/scheduler/interface/templates/scheduler/index.html 2011-04-07 04:01:20 +0000
+++ lava/scheduler/interface/templates/scheduler/index.html 2011-04-11 23:20:52 +0000
@@ -1,4 +1,4 @@
1{% include "base.html" %}1{% extends "base.html" %}
2{% block content %}2{% block content %}
3<form action="." method="post">3<form action="." method="post">
4 {{ form.as_p }}4 {{ form.as_p }}
@@ -30,3 +30,9 @@
30{% endif %}30{% endif %}
31</p>31</p>
32{% endblock %}32{% endblock %}
33{% block heading %}
34Submit test job
35{% endblock %}
36{% block title %}
37LAVA: Submit test job
38{% endblock %}
3339
=== modified file 'lava/scheduler/interface/urls.py'
--- lava/scheduler/interface/urls.py 2011-04-08 05:22:01 +0000
+++ lava/scheduler/interface/urls.py 2011-04-11 23:20:52 +0000
@@ -5,12 +5,11 @@
5admin.autodiscover()5admin.autodiscover()
66
7urlpatterns = patterns('',7urlpatterns = patterns('',
8 (r'^scheduler/', include('lava.scheduler.interface.scheduler_app.urls')),
9 (r'^openid/', include('django_openid_auth.urls')),8 (r'^openid/', include('django_openid_auth.urls')),
10 (r'^logout$', 'django.contrib.auth.views.logout'),9 (r'^logout$', 'django.contrib.auth.views.logout'),
11 # Uncomment the admin/doc line below and add 'django.contrib.admindocs' 10 # Uncomment the admin/doc line below and add 'django.contrib.admindocs'
12 # to INSTALLED_APPS to enable admin documentation:11 # to INSTALLED_APPS to enable admin documentation:
13 # (r'^admin/doc/', include('django.contrib.admindocs.urls')),12 # (r'^admin/doc/', include('django.contrib.admindocs.urls')),
14
15 (r'^admin/', include(admin.site.urls)),13 (r'^admin/', include(admin.site.urls)),
14 (r'^', include('lava.scheduler.interface.scheduler_app.urls')),
16)15)

Subscribers

People subscribed via source and target branches