Merge lp:~jak-o/beeseek/configuration-ui into lp:beeseek/1.0

Proposed by Jacopo Moretto
Status: Merged
Approved by: Andrea Corbellini
Approved revision: 246
Merged at revision: not available
Proposed branch: lp:~jak-o/beeseek/configuration-ui
Merge into: lp:beeseek/1.0
Diff against target: None lines
To merge this branch: bzr merge lp:~jak-o/beeseek/configuration-ui
Reviewer Review Type Date Requested Status
Andrea Corbellini Approve
Review via email: mp+5062@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Andrea Corbellini (andrea.corbellini) wrote :

Thanks for your work. The new UI looks really great, but I found a small
problem: when there are multiple options, the layout looks strange.
 review needsfixing

I'm attaching a patch so that you can add some dummy options to the
configuration page and do better tests. You can apply it with the
command:
 patch -p0 < add-dummy-options
(Remember not to commit "beeseek/applications/peer/main.py")

review: Needs Fixing
Revision history for this message
Andrea Corbellini (andrea.corbellini) wrote :

=== modified file 'beeseek/applications/peer/main.py'
--- beeseek/applications/peer/main.py 2009-03-27 12:53:24 +0000
+++ beeseek/applications/peer/main.py 2009-03-31 13:32:19 +0000
@@ -38,6 +38,23 @@
                           'Wheter to track SSL encoded pages.',
                           default=False)

+ test1 = Option(str, 'Test option 1', None, None,
+ 'This is a test option.',
+ default="bla bla bla")
+
+ class TestConfig(BaseOptionSection):
+
+ implements(IOptionSection)
+ help = 'This is a test section.'
+
+ test2 = Option(str, 'Test option 2 ', None, None,
+ 'This is a test option.',
+ default="bla bla bla")
+
+ test3 = Option(str, 'Test option 3', None, None,
+ 'This is a test option.',
+ default="bla bla bla")
+

 class ProxySession(BaseServerSession):

lp:~jak-o/beeseek/configuration-ui updated
246. By Jacopo Moretto

Minor adjustments to BeeSeek Peer configuration theme.

Revision history for this message
Andrea Corbellini (andrea.corbellini) wrote :

This new revision looks good. +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'webdata/css/style_basic.css'
2--- webdata/css/style_basic.css 2009-02-19 19:45:15 +0000
3+++ webdata/css/style_basic.css 2009-03-31 12:22:14 +0000
4@@ -182,3 +182,66 @@
5 div#top div#users {
6 float:left;
7 }
8+
9+form#options {
10+ position:relative;
11+ left:0;
12+ margin:-1em 0 0 0;
13+ width:100%;
14+}
15+
16+li.option {
17+ border-top: 1px dotted #cfcfcf;
18+ list-style-type: none;
19+ padding: 5px;
20+ width: 400px;
21+}
22+
23+li.option h3 {
24+ margin:0;
25+ padding:5px 0;
26+}
27+
28+li.option h3 a{
29+ text-decoration:none;
30+ color: #556d7a;
31+}
32+li.option p {
33+ font-size: 8pt;
34+ margin:0;
35+ padding:0 0 10px 0;
36+}
37+
38+div#save {
39+ position:relative;
40+ padding: 10px;
41+ border-top: 1px dotted #cfcfcf;
42+ margin-left:3em;
43+ width: 400px;
44+}
45+
46+a#cancel {
47+ background: #f5f5f5;
48+ border: 1px solid #cfcfcf;
49+ font-family: "Bitstream Vera Sans", "Trebuchet MS", Trebuchet, Verdana, Arial, Sans-serif;
50+ font-size: 9pt;
51+ color: #494949;
52+ text-decoration: none;
53+ padding: 1px 7px;
54+ margin: 0 10px;
55+}
56+
57+p#help {
58+ position:relative;
59+ border: 1px dotted #cfcfcf;
60+ padding:5px 7px;
61+ font-weight:bold;
62+ font-style: italic;
63+ width:350px;
64+ background: #f5f5f5;
65+}
66+
67+p#help a{
68+ text-decoration:none;
69+ color: #556d7a;
70+}
71\ No newline at end of file
72
73=== modified file 'webdata/html/config-bottom.html'
74--- webdata/html/config-bottom.html 2009-03-01 13:18:04 +0000
75+++ webdata/html/config-bottom.html 2009-03-31 12:22:14 +0000
76@@ -1,4 +1,4 @@
77 </ol>
78- <p>If you need help, <a href="http://wiki.beeseek.org/Documentation">read
79+ <p id="help">If you need help, <a href="http://wiki.beeseek.org/Documentation">read
80 the wiki</a> or <a href="http://wiki.beeseek.org/Feedback">contact the
81 team</a>.</p>
82
83=== modified file 'webdata/html/config-section.html'
84--- webdata/html/config-section.html 2009-03-03 12:49:17 +0000
85+++ webdata/html/config-section.html 2009-03-31 12:22:14 +0000
86@@ -1,2 +1,4 @@
87- <h3>$name <small><a href="/beeseek:configure/$lowername">edit</a></small></h3>
88- <p>$description</p>
89+ <li class="option">
90+ <h3>$name <small><a href="/beeseek:configure/$lowername">edit</a></small></h3>
91+ <p>$description</p>
92+ </li>
93
94=== modified file 'webdata/html/configopt-base.html'
95--- webdata/html/configopt-base.html 2009-03-19 17:10:53 +0000
96+++ webdata/html/configopt-base.html 2009-03-31 12:22:14 +0000
97@@ -1,3 +1,5 @@
98- <h3>$name</h3>
99- <p>$description</p>
100- $widget
101+ <li class="option">
102+ <h3>$name</h3>
103+ <p>$description</p>
104+ $widget
105+ </li>
106
107=== modified file 'webdata/html/configsec-bottom.html'
108--- webdata/html/configsec-bottom.html 2009-03-11 16:14:56 +0000
109+++ webdata/html/configsec-bottom.html 2009-03-31 12:22:14 +0000
110@@ -1,4 +1,6 @@
111 </ol>
112- <p><input type="submit" value="Save changes" /> or
113- <a href="/beeseek:configure">Cancel</a>.</p>
114+
115+ <div id="save">
116+ <input value="Save changes" type="submit"/> <a id="cancel" href="http://www.beeseek.org/beeseek:configure">Cancel</a>
117+ </div>
118 </form>
119
120=== modified file 'webdata/html/configsec-top.html'
121--- webdata/html/configsec-top.html 2009-03-20 16:23:12 +0000
122+++ webdata/html/configsec-top.html 2009-03-31 12:22:14 +0000
123@@ -1,4 +1,4 @@
124 <h2>Configuration for $instance:appname - $name</h2>
125 <p>$description</p>
126- <form action="" method="post">
127+ <form action="" method="post" id="options">
128 <ol>

Subscribers

People subscribed via source and target branches