Merge lp:~bac/launchpad/bug-419742 into lp:launchpad
Proposed by
Brad Crittenden
Status: | Merged |
---|---|
Approved by: | Brad Crittenden |
Approved revision: | no longer in the source branch. |
Merged at revision: | not available |
Proposed branch: | lp:~bac/launchpad/bug-419742 |
Merge into: | lp:launchpad |
Diff against target: |
242 lines 4 files modified
lib/canonical/launchpad/icing/style-3-0.css (+4/-0) lib/lp/registry/stories/product/xx-product-files.txt (+11/-10) lib/lp/registry/templates/product-files.pt (+57/-32) lib/lp/registry/templates/productrelease-portlet-data.pt (+6/-8) |
To merge this branch: | bzr merge lp:~bac/launchpad/bug-419742 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Albisetti (community) | ui | Approve | |
Paul Hummer (community) | Approve | ||
Review via email: mp+13807@code.launchpad.net |
Commit message
Update the +download page providing more information about releases and re-arranging to better present existing information.
To post a comment you must log in.
= Summary =
Bug 419742 suggested new information to present on the $product/+download page and
ways to make it more presentable.
== Proposed fix ==
Re-arrange lots of the data:
* Remove the extra information about verification and just present the help link.
* Move the release date into the header.
* Put 'release notes' and 'changelog' into a collapsible field called 'Release
information'.
* Add '(+) Add download file' under the release table.
* Keep the link at the end for each release but add the series name to it for
distinguishing it.[1]
* Make each release a portlet.[2]
[1] beuno had asked that this entire line be removed. I restored it because if a
release has no download files there is no place to add the '(+) Add download file'
menu link. Releases with no files are just skipped in the body of this page. So it
made sense to bring it back and just list all of the releases in one place. We often
get complaints from users that it is too hard to figure out where to go to add
download files I was hesitant to remove one place they may have grown used to using.
[2] I have made each use the 'top-portlet' class and conditionally insert the border.
This requires less code reorganization than conditionally choosing a class. TAL is
really bad when you need to select attributes conditionally, basically you have to
violate DRY all over the place.
== Pre-implementation notes ==
Lots of discussion with beuno. Unfortunately I have not been able to have a
discussion with mbp but hope to early next week to see if he still has any issues.
== Implementation details ==
As above.
== Tests ==
bin/test -vv -t xx-product- files.txt
== Demo and Q/A ==
https:/ /launchpad. dev/firefox/ +download
Also go to any other project and add some releases and download files.
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files: registry/ templates/ product- files.pt /launchpad/ icing/style- 3-0.css registry/ stories/ product/ xx-product- files.txt registry/ templates/ productrelease- portlet- data.pt
lib/lp/
lib/canonical
lib/lp/
lib/lp/