Merge lp:~sinzui/launchpad/source-package-ui into lp:launchpad
- source-package-ui
- Merge into devel
Proposed by
Curtis Hovey
Status: | Merged |
---|---|
Approved by: | Aaron Bentley |
Approved revision: | no longer in the source branch. |
Merged at revision: | not available |
Proposed branch: | lp:~sinzui/launchpad/source-package-ui |
Merge into: | lp:launchpad |
Diff against target: | None lines |
To merge this branch: | bzr merge lp:~sinzui/launchpad/source-package-ui |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Aaron Bentley (community) | code | Approve | |
Martin Albisetti (community) | ui | Approve | |
Review via email: mp+11718@code.launchpad.net |
Commit message
Description of the change
To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote : | # |
Revision history for this message
Martin Albisetti (beuno) wrote : | # |
For a mechanical change, this looks great.
It needs the h1, and there are a few comments on IRC that would be great to have.
review:
Approve
(ui)
Revision history for this message
Aaron Bentley (abentley) : | # |
review:
Approve
(code)
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1 | === modified file 'lib/canonical/launchpad/pagetitles.py' | |||
2 | --- lib/canonical/launchpad/pagetitles.py 2009-09-14 15:16:12 +0000 | |||
3 | +++ lib/canonical/launchpad/pagetitles.py 2009-09-14 18:06:44 +0000 | |||
4 | @@ -904,14 +904,6 @@ | |||
5 | 904 | 904 | ||
6 | 905 | sourcepackage_filebug = ContextTitle("Report a bug about %s") | 905 | sourcepackage_filebug = ContextTitle("Report a bug about %s") |
7 | 906 | 906 | ||
8 | 907 | sourcepackage_gethelp = ContextTitle('Help and support options for %s') | ||
9 | 908 | |||
10 | 909 | sourcepackage_packaging = ContextTitle('%s upstream links') | ||
11 | 910 | |||
12 | 911 | def sourcepackage_index(context, view): | ||
13 | 912 | """Return the page title for a source package in a distroseries.""" | ||
14 | 913 | return '%s source packages' % context.distroseries.title | ||
15 | 914 | |||
16 | 915 | sourcepackage_translate = ContextTitle('Help translate %s') | 907 | sourcepackage_translate = ContextTitle('Help translate %s') |
17 | 916 | 908 | ||
18 | 917 | sourcepackagenames_index = 'Source package name set' | 909 | sourcepackagenames_index = 'Source package name set' |
19 | 918 | 910 | ||
20 | === modified file 'lib/lp/answers/stories/question-add.txt' | |||
21 | --- lib/lp/answers/stories/question-add.txt 2009-09-09 06:37:22 +0000 | |||
22 | +++ lib/lp/answers/stories/question-add.txt 2009-09-13 20:51:35 +0000 | |||
23 | @@ -32,8 +32,9 @@ | |||
24 | 32 | ... 'http://launchpad.dev/ubuntu/hoary/' | 32 | ... 'http://launchpad.dev/ubuntu/hoary/' |
25 | 33 | ... '+sources/mozilla-firefox/+gethelp') | 33 | ... '+sources/mozilla-firefox/+gethelp') |
26 | 34 | >>> user_browser.title | 34 | >>> user_browser.title |
29 | 35 | 'Help and support options for ...mozilla-firefox...' | 35 | '+gethelp : ...mozilla-firefox... package : 5.04 : Ubuntu' |
30 | 36 | >>> user_browser.getLink('Submit your question').click() | 36 | |
31 | 37 | >>> user_browser.getLink('Ask a question').click() | ||
32 | 37 | >>> user_browser.title | 38 | >>> user_browser.title |
33 | 38 | '+addquestion : \xe2\x80\x9cmozilla-firefox\xe2\x80\x9d package | 39 | '+addquestion : \xe2\x80\x9cmozilla-firefox\xe2\x80\x9d package |
34 | 39 | : 5.04 : Ubuntu' | 40 | : 5.04 : Ubuntu' |
35 | 40 | 41 | ||
36 | === renamed file 'lib/lp/registry/templates/sourcepackage-gethelp.pt' => 'lib/lp/answers/templates/sourcepackage-gethelp.pt' | |||
37 | --- lib/lp/registry/templates/sourcepackage-gethelp.pt 2009-07-17 17:59:07 +0000 | |||
38 | +++ lib/lp/answers/templates/sourcepackage-gethelp.pt 2009-09-13 20:51:35 +0000 | |||
39 | @@ -3,10 +3,7 @@ | |||
40 | 3 | xmlns:tal="http://xml.zope.org/namespaces/tal" | 3 | xmlns:tal="http://xml.zope.org/namespaces/tal" |
41 | 4 | xmlns:metal="http://xml.zope.org/namespaces/metal" | 4 | xmlns:metal="http://xml.zope.org/namespaces/metal" |
42 | 5 | xmlns:i18n="http://xml.zope.org/namespaces/i18n" | 5 | xmlns:i18n="http://xml.zope.org/namespaces/i18n" |
47 | 6 | xml:lang="en" | 6 | metal:use-macro="view/macro:page/main_only" |
44 | 7 | lang="en" | ||
45 | 8 | dir="ltr" | ||
46 | 9 | metal:use-macro="context/@@main_template/master" | ||
48 | 10 | i18n:domain="launchpad" | 7 | i18n:domain="launchpad" |
49 | 11 | > | 8 | > |
50 | 12 | 9 | ||
51 | @@ -14,10 +11,9 @@ | |||
52 | 14 | 11 | ||
53 | 15 | <div metal:fill-slot="main"> | 12 | <div metal:fill-slot="main"> |
54 | 16 | 13 | ||
59 | 17 | <h1>Help for <span | 14 | <h1 tal:content="view/page_title" /> |
60 | 18 | tal:replace="context/displayname" /> </h1> | 15 | |
61 | 19 | 16 | <div class="top-portlet"> | |
58 | 20 | |||
62 | 21 | <h2>Ask your question!</h2> | 17 | <h2>Ask your question!</h2> |
63 | 22 | 18 | ||
64 | 23 | <p> | 19 | <p> |
65 | @@ -26,13 +22,15 @@ | |||
66 | 26 | modules/canonical.launchpad.webapp.vhosts/allvhosts/configs/answers/rooturl">Launchpad | 22 | modules/canonical.launchpad.webapp.vhosts/allvhosts/configs/answers/rooturl">Launchpad |
67 | 27 | answer</a> tracker. | 23 | answer</a> tracker. |
68 | 28 | </p> | 24 | </p> |
74 | 29 | <ul> | 25 | <ul class="horizontal"> |
75 | 30 | <li><a tal:attributes="href | 26 | <li><a class="sprite info" |
76 | 31 | string:${context/fmt:url}/+questions">Search through existing questions</a></li> | 27 | tal:attributes="href |
77 | 32 | <li><a tal:attributes="href | 28 | string:${context/fmt:url}/+questions">Search through existing questions</a></li> |
78 | 33 | string:${context/fmt:url}/+addquestion">Submit your question</a></li> | 29 | <li><a tal:replace="structure context/menu:answers/new/fmt:link" /></li> |
79 | 34 | </ul> | 30 | </ul> |
80 | 31 | </div> | ||
81 | 35 | 32 | ||
82 | 33 | <div> | ||
83 | 36 | <h2>Other support options</h2> | 34 | <h2>Other support options</h2> |
84 | 37 | 35 | ||
85 | 38 | <p> | 36 | <p> |
86 | @@ -63,7 +61,7 @@ | |||
87 | 63 | Thanks! | 61 | Thanks! |
88 | 64 | </p> | 62 | </p> |
89 | 65 | </tal:needupstream> | 63 | </tal:needupstream> |
91 | 66 | 64 | </div> | |
92 | 67 | </div> | 65 | </div> |
93 | 68 | </body> | 66 | </body> |
94 | 69 | </html> | 67 | </html> |
95 | 70 | 68 | ||
96 | === modified file 'lib/lp/registry/browser/configure.zcml' | |||
97 | --- lib/lp/registry/browser/configure.zcml 2009-09-14 15:16:12 +0000 | |||
98 | +++ lib/lp/registry/browser/configure.zcml 2009-09-14 18:06:44 +0000 | |||
99 | @@ -1994,20 +1994,26 @@ | |||
100 | 1994 | name="+index" | 1994 | name="+index" |
101 | 1995 | facet="overview" | 1995 | facet="overview" |
102 | 1996 | template="../templates/sourcepackage-index.pt"/> | 1996 | template="../templates/sourcepackage-index.pt"/> |
103 | 1997 | <browser:page | ||
104 | 1998 | name="+gethelp" | ||
105 | 1999 | facet="answers" | ||
106 | 2000 | template="../templates/sourcepackage-gethelp.pt"/> | ||
107 | 2001 | <browser:page | ||
108 | 2002 | name="+packaging" | ||
109 | 2003 | facet="overview" | ||
110 | 2004 | template="../templates/sourcepackage-packaging.pt"/> | ||
111 | 2005 | <!-- SourcePackage Portlets --> | 1997 | <!-- SourcePackage Portlets --> |
112 | 2006 | <browser:page | 1998 | <browser:page |
113 | 2007 | name="+portlet-details" | 1999 | name="+portlet-details" |
114 | 2008 | facet="overview" | 2000 | facet="overview" |
115 | 2009 | template="../templates/sourcepackage-portlet-details.pt"/> | 2001 | template="../templates/sourcepackage-portlet-details.pt"/> |
116 | 2010 | </browser:pages> | 2002 | </browser:pages> |
117 | 2003 | <browser:page | ||
118 | 2004 | for="lp.registry.interfaces.sourcepackage.ISourcePackage" | ||
119 | 2005 | permission="zope.Public" | ||
120 | 2006 | class="lp.registry.browser.sourcepackage.SourcePackagePackaging" | ||
121 | 2007 | name="+packaging" | ||
122 | 2008 | facet="overview" | ||
123 | 2009 | template="../templates/sourcepackage-packaging.pt"/> | ||
124 | 2010 | <browser:page | ||
125 | 2011 | for="lp.registry.interfaces.sourcepackage.ISourcePackage" | ||
126 | 2012 | permission="zope.Public" | ||
127 | 2013 | class="lp.registry.browser.sourcepackage.SourcePackageHelpView" | ||
128 | 2014 | name="+gethelp" | ||
129 | 2015 | facet="answers" | ||
130 | 2016 | template="../../answers/templates/sourcepackage-gethelp.pt"/> | ||
131 | 2011 | <browser:menus | 2017 | <browser:menus |
132 | 2012 | classes=" | 2018 | classes=" |
133 | 2013 | SourcePackageFacets | 2019 | SourcePackageFacets |
134 | 2014 | 2020 | ||
135 | === modified file 'lib/lp/registry/browser/sourcepackage.py' | |||
136 | --- lib/lp/registry/browser/sourcepackage.py 2009-09-04 12:32:58 +0000 | |||
137 | +++ lib/lp/registry/browser/sourcepackage.py 2009-09-14 04:24:20 +0000 | |||
138 | @@ -9,7 +9,9 @@ | |||
139 | 9 | 'SourcePackageBreadcrumb', | 9 | 'SourcePackageBreadcrumb', |
140 | 10 | 'SourcePackageChangeUpstreamView', | 10 | 'SourcePackageChangeUpstreamView', |
141 | 11 | 'SourcePackageFacets', | 11 | 'SourcePackageFacets', |
142 | 12 | 'SourcePackageHelpView', | ||
143 | 12 | 'SourcePackageNavigation', | 13 | 'SourcePackageNavigation', |
144 | 14 | 'SourcePackagePackaging', | ||
145 | 13 | 'SourcePackageView', | 15 | 'SourcePackageView', |
146 | 14 | ] | 16 | ] |
147 | 15 | 17 | ||
148 | @@ -157,6 +159,11 @@ | |||
149 | 157 | self.processForm() | 159 | self.processForm() |
150 | 158 | 160 | ||
151 | 159 | @property | 161 | @property |
152 | 162 | def page_title(self): | ||
153 | 163 | """The HTML page title.""" | ||
154 | 164 | return '%s package' % self.context.name | ||
155 | 165 | |||
156 | 166 | @property | ||
157 | 160 | def cancel_url(self): | 167 | def cancel_url(self): |
158 | 161 | return canonical_url(self.context) | 168 | return canonical_url(self.context) |
159 | 162 | 169 | ||
160 | @@ -245,3 +252,21 @@ | |||
161 | 245 | @property | 252 | @property |
162 | 246 | def potemplates(self): | 253 | def potemplates(self): |
163 | 247 | return list(self.context.getCurrentTranslationTemplates()) | 254 | return list(self.context.getCurrentTranslationTemplates()) |
164 | 255 | |||
165 | 256 | |||
166 | 257 | class SourcePackagePackaging: | ||
167 | 258 | """A View to show where the package is packged.""" | ||
168 | 259 | |||
169 | 260 | @property | ||
170 | 261 | def page_title(self): | ||
171 | 262 | """The HTML page title.""" | ||
172 | 263 | return 'Upstream links' | ||
173 | 264 | |||
174 | 265 | |||
175 | 266 | class SourcePackageHelpView: | ||
176 | 267 | """A View to show Answers help.""" | ||
177 | 268 | |||
178 | 269 | @property | ||
179 | 270 | def page_title(self): | ||
180 | 271 | """The HTML page title.""" | ||
181 | 272 | return 'Help and support options' | ||
182 | 248 | 273 | ||
183 | === modified file 'lib/lp/registry/templates/sourcepackage-index.pt' | |||
184 | --- lib/lp/registry/templates/sourcepackage-index.pt 2009-08-07 22:14:06 +0000 | |||
185 | +++ lib/lp/registry/templates/sourcepackage-index.pt 2009-09-14 04:24:20 +0000 | |||
186 | @@ -3,148 +3,180 @@ | |||
187 | 3 | xmlns:tal="http://xml.zope.org/namespaces/tal" | 3 | xmlns:tal="http://xml.zope.org/namespaces/tal" |
188 | 4 | xmlns:metal="http://xml.zope.org/namespaces/metal" | 4 | xmlns:metal="http://xml.zope.org/namespaces/metal" |
189 | 5 | xmlns:i18n="http://xml.zope.org/namespaces/i18n" | 5 | xmlns:i18n="http://xml.zope.org/namespaces/i18n" |
196 | 6 | xml:lang="en" | 6 | metal:use-macro="view/macro:page/main_only" |
197 | 7 | lang="en" | 7 | i18n:domain="launchpad"> |
192 | 8 | dir="ltr" | ||
193 | 9 | metal:use-macro="context/@@main_template/master" | ||
194 | 10 | i18n:domain="launchpad" | ||
195 | 11 | > | ||
198 | 12 | <body> | 8 | <body> |
235 | 13 | <metal:heading fill-slot="pageheading"> | 9 | <div metal:fill-slot="main"> |
236 | 14 | <h1 tal:content="context/title">source package name</h1> | 10 | <div class="top-portlet"> |
237 | 15 | </metal:heading> | 11 | <h2>Versions published</h2> |
238 | 16 | 12 | ||
239 | 17 | <metal:leftportlets fill-slot="portlets_one"> | 13 | <p class="warning message" tal:condition="not:context/currentrelease"> |
240 | 18 | <div tal:replace="structure context/@@+portlet-details" /> | 14 | There is no current release of this source package |
241 | 19 | </metal:leftportlets> | 15 | in <span tal:replace="context/distroseries/title" />. |
242 | 20 | 16 | You can still report bugs, make translations, and so on, | |
243 | 21 | <metal:rightportlets fill-slot="portlets_two"> | 17 | but they might not be used until the package is published. |
244 | 22 | <div tal:replace="structure context/@@+portlet-releases" /> | 18 | </p> |
245 | 23 | <div tal:replace="structure context/@@+portlet-upstream" /> | 19 | |
246 | 24 | </metal:rightportlets> | 20 | <dl id="publishing_history" tal:condition="view/published_by_pocket"> |
247 | 25 | 21 | <tal:block repeat="pocket view/published_by_pocket"> | |
248 | 26 | <div metal:fill-slot="main"> | 22 | <tal:block condition="pocket/packages"> |
249 | 27 | 23 | <dt><dfn tal:content="pocket/pocketdetails/title">Plain</dfn></dt> | |
250 | 28 | <h2>Versions published</h2> | 24 | <dd> |
251 | 29 | 25 | <p tal:content="pocket/pocketdetails/description"> | |
252 | 30 | <p class="warning message" tal:condition="not:context/currentrelease"> | 26 | The description of the pocket. |
253 | 31 | There is no current release of this source package | 27 | </p> |
254 | 32 | in <span tal:replace="context/distroseries/title" />. | 28 | <ul> |
255 | 33 | You can still report bugs, make translations, and so on, | 29 | <li tal:repeat="spr pocket/packages"> |
256 | 34 | but they might not be used until the package is published. | 30 | <a tal:content="spr/displayname" |
257 | 35 | </p> | 31 | tal:attributes="href spr/fmt:url">apache2</a> |
258 | 36 | 32 | </li> | |
259 | 37 | <dl id="publishing_history" tal:condition="view/published_by_pocket"> | 33 | </ul> |
260 | 38 | <tal:block repeat="pocket view/published_by_pocket"> | 34 | </dd> |
261 | 39 | <tal:block condition="pocket/packages"> | 35 | </tal:block> |
262 | 40 | <dt><dfn tal:content="pocket/pocketdetails/title">Plain</dfn></dt> | 36 | </tal:block> |
263 | 41 | <dd> | 37 | </dl> |
264 | 42 | <p tal:content="pocket/pocketdetails/description"> | 38 | |
265 | 43 | The description of the pocket. | 39 | <ul class="horizontal"> |
266 | 44 | </p> | 40 | <li> |
267 | 45 | <ul> | 41 | <a tal:replace="structure context/menu:overview/changelog/fmt:link" /> |
268 | 46 | <li tal:repeat="spr pocket/packages"> | 42 | </li> |
269 | 47 | <a tal:content="spr/displayname" | 43 | </ul> |
270 | 48 | tal:attributes="href spr/fmt:url">apache2</a> | 44 | </div> |
271 | 45 | |||
272 | 46 | <div class="yui-g"> | ||
273 | 47 | <div class="yui-u first"> | ||
274 | 48 | <div tal:replace="structure context/@@+portlet-details" /> | ||
275 | 49 | </div> | ||
276 | 50 | |||
277 | 51 | <div class="yui-u"> | ||
278 | 52 | <div class="portlet"> | ||
279 | 53 | <h2>Upstream associations</h2> | ||
280 | 54 | |||
281 | 55 | <div id="upstreams" class="two-column-list" | ||
282 | 56 | tal:define="series context/productseries"> | ||
283 | 57 | <tal:has_series condition="series"> | ||
284 | 58 | <dl | ||
285 | 59 | tal:define="project series/product/project" | ||
286 | 60 | tal:condition="project"> | ||
287 | 61 | <dt>Project Group:</dt> | ||
288 | 62 | <dd> | ||
289 | 63 | <a tal:replace="structure project/fmt:link" /> | ||
290 | 64 | </dd> | ||
291 | 65 | </dl> | ||
292 | 66 | |||
293 | 67 | <dl> | ||
294 | 68 | <dt>Project:</dt> | ||
295 | 69 | <dd> | ||
296 | 70 | <a tal:replace="structure series/product/fmt:link" /> | ||
297 | 71 | </dd> | ||
298 | 72 | </dl> | ||
299 | 73 | |||
300 | 74 | <dl> | ||
301 | 75 | <dt>Branch:</dt> | ||
302 | 76 | <dd> | ||
303 | 77 | <a | ||
304 | 78 | tal:content="series/name" | ||
305 | 79 | tal:attributes="href series/fmt:url">Series</a> | ||
306 | 80 | </dd> | ||
307 | 81 | </dl> | ||
308 | 82 | </tal:has_series> | ||
309 | 83 | |||
310 | 84 | <tal:has_no_series condition="not: context/productseries"> | ||
311 | 85 | <dl> | ||
312 | 86 | <dt>Related project:</dt> | ||
313 | 87 | <dd> | ||
314 | 88 | <em>No project associated with this sourcepackage</em> | ||
315 | 89 | </dd> | ||
316 | 90 | </dl> | ||
317 | 91 | </tal:has_no_series> | ||
318 | 92 | </div> | ||
319 | 93 | |||
320 | 94 | <ul class="horizontal"> | ||
321 | 95 | <li> | ||
322 | 96 | <a tal:replace="structure context/menu:overview/edit_packaging/fmt:link" /> | ||
323 | 97 | </li> | ||
324 | 98 | <li> | ||
325 | 99 | <a tal:replace="structure context/menu:overview/packaging/fmt:link" /> | ||
326 | 100 | </li> | ||
327 | 101 | </ul> | ||
328 | 102 | </div> | ||
329 | 103 | </div> | ||
330 | 104 | </div> | ||
331 | 105 | |||
332 | 106 | <div class="yui-g"> | ||
333 | 107 | <div class="yui-u first"> | ||
334 | 108 | <div tal:replace="structure context/@@+portlet-releases" /> | ||
335 | 109 | |||
336 | 110 | <ul class="horizontal" style="margin-bottom: 1em;"> | ||
337 | 111 | <li> | ||
338 | 112 | <a tal:replace="structure context/menu:overview/builds/fmt:link" /> | ||
339 | 49 | </li> | 113 | </li> |
340 | 50 | </ul> | 114 | </ul> |
402 | 51 | </dd> | 115 | </div> |
403 | 52 | </tal:block> | 116 | |
404 | 53 | </tal:block> | 117 | <div class="yui-u"> |
405 | 54 | </dl> | 118 | <div tal:replace="structure context/@@+portlet-upstream" /> |
406 | 55 | 119 | ||
407 | 56 | <h2>Upstream associations</h2> | 120 | <div> |
408 | 57 | 121 | <h2>Binary packages</h2> | |
409 | 58 | <ul id="upstreams" tal:define="series context/productseries"> | 122 | |
410 | 59 | <tal:has_series condition="series"> | 123 | <div id="binaries" tal:define="binaries view/binaries"> |
411 | 60 | <li tal:condition="series/product/project"> | 124 | <ul tal:condition="binaries"> |
412 | 61 | <b>Project Group:</b> | 125 | <li tal:repeat="bin_name binaries"> |
413 | 62 | <a tal:define="project series/product/project" | 126 | <span tal:replace="bin_name" /> |
414 | 63 | tal:content="project/displayname" | 127 | <span tal:repeat="archtag python:binaries[bin_name]"> |
415 | 64 | tal:attributes="href string:/${project/name}"> | 128 | (<a tal:content="archtag" tal:attributes="href |
416 | 65 | Project</a> | 129 | string:../../${archtag}/${bin_name}" />)</span> |
417 | 66 | </li> | 130 | </li> |
418 | 67 | <li><b>Project:</b> | 131 | </ul> |
419 | 68 | <a tal:condition="series/product" | 132 | |
420 | 69 | tal:content="series/product/displayname" | 133 | <p tal:condition="not: binaries"> |
421 | 70 | tal:attributes="href string:/${series/product/name}"> | 134 | No binaries have been generated for this release. |
422 | 71 | Project</a> | 135 | </p> |
423 | 72 | </li> | 136 | </div> |
424 | 73 | <li><b>Branch: </b> | 137 | </div> |
425 | 74 | <a tal:define="productseries context/productseries" | 138 | </div> |
426 | 75 | tal:content="productseries/name" | 139 | </div> |
427 | 76 | tal:attributes="href productseries/fmt:url"> | 140 | |
428 | 77 | ProjectSeries</a><br /> | 141 | <div class="portlet" |
429 | 78 | </li> | 142 | tal:define="current context/currentrelease" |
430 | 79 | </tal:has_series> | 143 | tal:condition="current"> |
431 | 80 | <tal:has_no_series condition="not: context/productseries"> | 144 | <h2>Download files from current release (<span |
432 | 81 | <li><b>Related project:</b> | 145 | tal:replace="current/version">2.1.3-4</span>)</h2> |
433 | 82 | <i> | 146 | |
434 | 83 | No project associated with this sourcepackage | 147 | <div id="files" tal:content="structure current/@@+files" /> |
435 | 84 | </i> | 148 | |
436 | 85 | </li> | 149 | <h2>Package relationships</h2> |
437 | 86 | </tal:has_no_series> | 150 | |
438 | 87 | <small>(<a tal:attributes="href string:${context/fmt:url}/+packaging">update</a>)</small> | 151 | <table width="100%" valign="top"> |
439 | 88 | </ul> | 152 | <tr> |
440 | 89 | 153 | <td id="depends" tal:define="relationships view/builddepends"> | |
441 | 90 | <tal:block tal:condition="context/currentrelease" | 154 | <h3>Build dependencies</h3> |
442 | 91 | tal:define="current context/currentrelease"> | 155 | <tal:block tal:condition="relationships/has_items" |
443 | 92 | 156 | replace="structure relationships/@@+render-list"/> | |
444 | 93 | <h2>Download files from current release (<span | 157 | </td> |
445 | 94 | tal:replace="current/version">2.1.3-4</span>)</h2> | 158 | <td id="dependsindep" |
446 | 95 | 159 | tal:define="relationships view/builddependsindep"> | |
447 | 96 | <div id="files" tal:content="structure current/@@+files" /> | 160 | <h3>Platform-independent build dependencies</h3> |
448 | 97 | 161 | <tal:block tal:condition="relationships/has_items" | |
449 | 98 | <h2>Binary packages</h2> | 162 | replace="structure relationships/@@+render-list"/> |
450 | 99 | 163 | </td> | |
451 | 100 | <div id="binaries" tal:define="binaries view/binaries"> | 164 | </tr> |
452 | 101 | <ul tal:condition="binaries"> | 165 | <tr> |
453 | 102 | <li tal:repeat="bin_name binaries"> | 166 | <td id="conflicts" tal:define="relationships view/build_conflicts"> |
454 | 103 | <span tal:replace="bin_name" /> | 167 | <h3>Build conflicts</h3> |
455 | 104 | <span tal:repeat="archtag python:binaries[bin_name]"> | 168 | <tal:block tal:condition="relationships/has_items" |
456 | 105 | (<a tal:content="archtag" tal:attributes="href | 169 | replace="structure relationships/@@+render-list"/> |
457 | 106 | string:../../${archtag}/${bin_name}" />)</span> | 170 | </td> |
458 | 107 | </li> | 171 | <td id="conflictsindep" |
459 | 108 | </ul> | 172 | tal:define="relationships view/build_conflicts_indep"> |
460 | 109 | <p tal:condition="not: binaries"> | 173 | <h3>Platform-independent build dependencies</h3> |
461 | 110 | No binaries have been generated for this release. | 174 | <tal:block tal:condition="relationships/has_items" |
462 | 111 | </p> | 175 | replace="structure relationships/@@+render-list"/> |
463 | 176 | </td> | ||
464 | 177 | </tr> | ||
465 | 178 | </table> | ||
466 | 179 | </div> | ||
467 | 112 | </div> | 180 | </div> |
504 | 113 | 181 | </body> | |
469 | 114 | <h2>Package relationships</h2> | ||
470 | 115 | |||
471 | 116 | <table width="100%" valign="top"> | ||
472 | 117 | <tr> | ||
473 | 118 | <td id="depends" tal:define="relationships view/builddepends"> | ||
474 | 119 | <h3>Build dependencies</h3> | ||
475 | 120 | <tal:block tal:condition="relationships/has_items" | ||
476 | 121 | replace="structure relationships/@@+render-list"/> | ||
477 | 122 | </td> | ||
478 | 123 | <td id="dependsindep" | ||
479 | 124 | tal:define="relationships view/builddependsindep"> | ||
480 | 125 | <h3>Platform-independent build dependencies</h3> | ||
481 | 126 | <tal:block tal:condition="relationships/has_items" | ||
482 | 127 | replace="structure relationships/@@+render-list"/> | ||
483 | 128 | </td> | ||
484 | 129 | </tr> | ||
485 | 130 | <tr> | ||
486 | 131 | <td id="conflicts" tal:define="relationships view/build_conflicts"> | ||
487 | 132 | <h3>Build conflicts</h3> | ||
488 | 133 | <tal:block tal:condition="relationships/has_items" | ||
489 | 134 | replace="structure relationships/@@+render-list"/> | ||
490 | 135 | </td> | ||
491 | 136 | <td id="conflictsindep" | ||
492 | 137 | tal:define="relationships view/build_conflicts_indep"> | ||
493 | 138 | <h3>Platform-independent build dependencies</h3> | ||
494 | 139 | <tal:block tal:condition="relationships/has_items" | ||
495 | 140 | replace="structure relationships/@@+render-list"/> | ||
496 | 141 | </td> | ||
497 | 142 | </tr> | ||
498 | 143 | </table> | ||
499 | 144 | |||
500 | 145 | </tal:block> | ||
501 | 146 | |||
502 | 147 | </div> | ||
503 | 148 | </body> | ||
505 | 149 | </html> | 182 | </html> |
506 | 150 | |||
507 | 151 | 183 | ||
508 | === modified file 'lib/lp/registry/templates/sourcepackage-packaging.pt' | |||
509 | --- lib/lp/registry/templates/sourcepackage-packaging.pt 2009-09-07 09:15:02 +0000 | |||
510 | +++ lib/lp/registry/templates/sourcepackage-packaging.pt 2009-09-13 21:40:53 +0000 | |||
511 | @@ -3,24 +3,15 @@ | |||
512 | 3 | xmlns:tal="http://xml.zope.org/namespaces/tal" | 3 | xmlns:tal="http://xml.zope.org/namespaces/tal" |
513 | 4 | xmlns:metal="http://xml.zope.org/namespaces/metal" | 4 | xmlns:metal="http://xml.zope.org/namespaces/metal" |
514 | 5 | xmlns:i18n="http://xml.zope.org/namespaces/i18n" | 5 | xmlns:i18n="http://xml.zope.org/namespaces/i18n" |
519 | 6 | xml:lang="en" | 6 | metal:use-macro="view/macro:page/main_only" |
516 | 7 | lang="en" | ||
517 | 8 | dir="ltr" | ||
518 | 9 | metal:use-macro="context/@@main_template/master" | ||
520 | 10 | i18n:domain="launchpad" | 7 | i18n:domain="launchpad" |
521 | 11 | > | 8 | > |
522 | 12 | 9 | ||
523 | 13 | <body> | 10 | <body> |
524 | 14 | 11 | ||
525 | 15 | <metal:rightportlets fill-slot="portlets_two"> | ||
526 | 16 | <div tal:replace="structure context/@@+portlet-upstream" /> | ||
527 | 17 | </metal:rightportlets> | ||
528 | 18 | |||
529 | 19 | |||
530 | 20 | <div metal:fill-slot="main"> | 12 | <div metal:fill-slot="main"> |
531 | 21 | 13 | ||
534 | 22 | <h1>Upstream links for <span tal:replace="context/displayname">foobar in | 14 | <h1 tal:content="view/page_title" /> |
533 | 23 | hoary</span></h1> | ||
535 | 24 | 15 | ||
536 | 25 | <p> | 16 | <p> |
537 | 26 | Linking distribution packages to upstream series allows | 17 | Linking distribution packages to upstream series allows |
538 | @@ -53,14 +44,14 @@ | |||
539 | 53 | <tr tal:attributes=" | 44 | <tr tal:attributes=" |
540 | 54 | class python: packaging == context.packaging and 'highlight' or None"> | 45 | class python: packaging == context.packaging and 'highlight' or None"> |
541 | 55 | 46 | ||
550 | 56 | <td tal:content="packaging/distroseries/distribution/displayname"> | 47 | <td> |
551 | 57 | ubuntu | 48 | <a tal:replace="structure packaging/distroseries/distribution/fmt:link" /> |
552 | 58 | </td> | 49 | </td> |
553 | 59 | <td tal:content="packaging/distroseries/displayname"> | 50 | <td> |
554 | 60 | hoary | 51 | <a tal:replace="structure packaging/distroseries/fmt:link" /> |
555 | 61 | </td> | 52 | </td> |
556 | 62 | <td tal:content="packaging/productseries/title"> | 53 | <td> |
557 | 63 | evolution main | 54 | <a tal:replace="structure packaging/productseries/fmt:link" /> |
558 | 64 | </td> | 55 | </td> |
559 | 65 | 56 | ||
560 | 66 | </tr> | 57 | </tr> |
561 | 67 | 58 | ||
562 | === modified file 'lib/lp/registry/templates/sourcepackage-portlet-details.pt' | |||
563 | --- lib/lp/registry/templates/sourcepackage-portlet-details.pt 2009-07-17 17:59:07 +0000 | |||
564 | +++ lib/lp/registry/templates/sourcepackage-portlet-details.pt 2009-09-14 04:24:20 +0000 | |||
565 | @@ -5,30 +5,33 @@ | |||
566 | 5 | omit-tag=""> | 5 | omit-tag=""> |
567 | 6 | 6 | ||
568 | 7 | <div class="portlet" id="portlet-details"> | 7 | <div class="portlet" id="portlet-details"> |
588 | 8 | <h2><span tal:replace="context/name" /> details</h2> | 8 | <h2><span tal:replace="context/name" /> information</h2> |
589 | 9 | 9 | ||
590 | 10 | <div class="portletBody portletContent"> | 10 | <div class="two-column-list" |
591 | 11 | <tal:currentrelease tal:condition="context/currentrelease" | 11 | tal:define="current context/currentrelease"> |
592 | 12 | tal:define="current context/currentrelease"> | 12 | <tal:currentrelease tal:condition="current"> |
593 | 13 | <b>Current version:</b> | 13 | <dl> |
594 | 14 | <span tal:replace="current/version" /><br /> | 14 | <dt>Current version:</dt> |
595 | 15 | 15 | <dd><span tal:replace="current/version" /></dd> | |
596 | 16 | <b>Uploaded:</b> | 16 | </dl> |
597 | 17 | <span | 17 | <dl> |
598 | 18 | tal:attributes="title current/dateuploaded/fmt:datetime" | 18 | <dt>Uploaded:</dt> |
599 | 19 | tal:content="current/dateuploaded/fmt:approximatedate" /><br /> | 19 | <dd><span |
600 | 20 | 20 | tal:attributes="title current/dateuploaded/fmt:datetime" | |
601 | 21 | <b>Urgency:</b> | 21 | tal:content="current/dateuploaded/fmt:approximatedate" /></dd> |
602 | 22 | <span tal:content="current/urgency/title" /><br /> | 22 | </dl> |
603 | 23 | 23 | <dl> | |
604 | 24 | <b>Component:</b> | 24 | <dt>Urgency:</dt> |
605 | 25 | <span tal:content="current/component/name"/><br /> | 25 | <dd><span tal:content="current/urgency/title" /></dd> |
606 | 26 | 26 | </dl> | |
607 | 27 | <dl> | ||
608 | 28 | <dt>Component:</dt> | ||
609 | 29 | <dd><span tal:content="current/component/name"/></dd> | ||
610 | 30 | </dl> | ||
611 | 27 | </tal:currentrelease> | 31 | </tal:currentrelease> |
616 | 28 | <tal:norelease tal:condition="not: context/currentrelease"> | 32 | <tal:norelease tal:condition="not: current"> |
617 | 29 | <i>No published releases of this package have been uploaded in | 33 | <em>No published releases of this package have been uploaded in |
618 | 30 | <tal:block content="context/distroseries/fullseriesname" /> | 34 | <tal:series content="context/distroseries/fullseriesname" /></em> |
615 | 31 | </i> | ||
619 | 32 | </tal:norelease> | 35 | </tal:norelease> |
620 | 33 | </div> | 36 | </div> |
621 | 34 | 37 | ||
622 | 35 | 38 | ||
623 | === modified file 'lib/lp/soyuz/stories/soyuz/xx-distroseries-sources.txt' | |||
624 | --- lib/lp/soyuz/stories/soyuz/xx-distroseries-sources.txt 2009-09-07 13:26:02 +0000 | |||
625 | +++ lib/lp/soyuz/stories/soyuz/xx-distroseries-sources.txt 2009-09-14 04:24:20 +0000 | |||
626 | @@ -81,20 +81,10 @@ | |||
627 | 81 | It also provides the upstream relationships for this source, Project, | 81 | It also provides the upstream relationships for this source, Project, |
628 | 82 | Product and Branches: | 82 | Product and Branches: |
629 | 83 | 83 | ||
644 | 84 | >>> print find_tag_by_id(browser.contents, 'upstreams') | 84 | >>> print extract_text(find_tag_by_id(browser.contents, 'upstreams')) |
645 | 85 | <ul id="upstreams"> | 85 | Project Group: the Mozilla Project |
646 | 86 | <li> | 86 | Project: Mozilla Firefox ... |
647 | 87 | <b>Project Group:</b> | 87 | Branch: trunk |
634 | 88 | <a href="/mozilla">the Mozilla Project</a> | ||
635 | 89 | </li> | ||
636 | 90 | <li><b>Project:</b> | ||
637 | 91 | <a href="/firefox">Mozilla Firefox</a> | ||
638 | 92 | </li> | ||
639 | 93 | <li><b>Branch: </b> | ||
640 | 94 | <a href="/firefox/trunk">trunk</a><br /> | ||
641 | 95 | </li> | ||
642 | 96 | <small>(<a href="/ubuntu/warty/+source/mozilla-firefox/+packaging">update</a>)</small> | ||
643 | 97 | </ul> | ||
648 | 98 | 88 | ||
649 | 99 | The user can also download the files for the "currentrelease" (last | 89 | The user can also download the files for the "currentrelease" (last |
650 | 100 | published version) if they are available: | 90 | published version) if they are available: |
651 | 101 | 91 |
This is my branch to update the last source package pages to UI 3.0.
lp:~sinzui/launchpad/source-package-ui /bugs.launchpad .net/bugs/ 429020 e-packaging. txt \ -sources implementation: no one
Diff size: 651
Launchpad bug: https:/
Test command: ./bin/test -vv \
-t question-add.txt \
-t xx-sourcepackag
-t xx-distroseries
Pre-
Target release: 3.1.0
= Update the last source package pages to UI 3.0 =
sourcepackage- gethelp. pt
* Move the template to answers. Update it to 3.0.
sourcepackage- packaging. pt
* Linkify the items in the table so that the portlets are not needed.
* Update the page to UI 3.0
sourcepackage- index.pt
* Move the details portlet to the top of the first column, Move Upstream
associations to the top right column.
* Move Releases in Ubuntu and Upstream release series inline.
* Move Changelog, Show builds, and Show upstream links into content
* Change upstream link and Set upstream link are identical; merge them.
* I considered the moving the download files to the side portlets, but
their purpose is not really for users, be developers, so I think they
belong on the content.
== QA ==
Compare people. canonical. com/~curtis/ sourcepackage. png /edge.launchpad .net/ubuntu/ karmic/ +source/ alsa-utils
http://
With
https:/
== Lint ==
Linting changed files: /launchpad/ pagetitles. py answers/ stories/ question- add.txt answers/ templates/ sourcepackage- gethelp. pt registry/ browser/ configure. zcml registry/ browser/ sourcepackage. py registry/ templates/ sourcepackage- index.pt registry/ templates/ sourcepackage- packaging. pt registry/ templates/ sourcepackage- portlet- details. pt soyuz/stories/ soyuz/xx- distroseries- sources. txt
lib/canonical
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
== Test ==
* lib/lp/ answers/ stories/ question- add.txt soyuz/stories/ soyuz/xx- distroseries- sources. txt
* Added a test to verify that the breadcrumb shows up in the page title.
This is a known issue that Salgado has a fix for. It is out of scope
for this branch.
* Updated test to show the page uses the standard link.
* lib/lp/
* Updated brittle test.
== Implementation ==
* lib/canonical/ launchpad/ pagetitles. py answers/ templates/ sourcepackage- gethelp. pt registry/ browser/ configure. zcml registry/ browser/ sourcepackage. py lpView and SourcePackagePa ckaging to provide registry/ templates/ sourcepackage- index.pt registry/ template. ..
* Removed unused page titles.
* lib/lp/
* Moved page to answers.
* Updated the page to 3.0
* Switched to standard links.
* lib/lp/
* Updated config to use the template from the lp/ansers
* Registered +packaging as its own view.
* lib/lp/
* Added SourcePackageHe
unique heading and titles to the pages.
* lib/lp/
* Moved a lot of content around, and changed a many <ul> in <dl>.
But as you can see from the one test change, the content has not
changed at all. If the one test was not checking markup, there would
have been no failures.
* See the picture if you do not want to run the server.
* lib/lp/