Merge lp:~adiroiban/launchpad/bug-340664 into lp:launchpad

[Adi Roiban]

[Adi Roiban (adiroiban)]

= Bug 340664 =
As discussed during Translations 3.0 UI sprint, we've decided to add administration page listing all templates for a single series. This would be mostly used by maintainers, and initially should have following columns:

  * template
  * length (number of messages)
  * languages (that have at least some translations)
  * last update
  * editing button

Showing priority might not be a bad idea either.

== Proposed fix ==

Add priority, length and languages to the +templates pages.

== Pre-implementation notes ==
Last update and editing buttons were added in previous branches.

I talked with Danilo and he said the screenshots are OK and this is the intended way for fixing the bug.

== Implementation details ==
distroseries-templates and productseries-templates stories were merged into a single series-templates file.

Since both product and distribution series are using the same template for displaying templates list, I have create a base class for those views.

object-template.pt logic was moved from the template to the view and the new template should be easier to read.

== Tests ==
lp-test -t series-templates

== Demo and Q/A ==
Log in as admin and go to a product +templates page
https://translations.launchpad.dev/evolution/trunk/+templates

The table should look like in this screenshot:
http://launchpadlibrarian.net/37268665/productseries.png

Log in as admin and go to a distribution +templates page
The table should look like in this screenshot:
http://launchpadlibrarian.net/37268672/distroseries.png

When logged in as a normal user you should not see the edit, upload and admin links.

= Launchpad lint =

Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.

Linting changed files:
  lib/lp/translations/browser/distroseries.py
  lib/lp/translations/browser/productseries.py

[Michael Nelson (michael.nelson)]

Hi Adi, I'm requesting a UI review from Curtis first (he's been starved of them), as it's always worth doing that before the code review.

[Curtis Hovey (sinzui)]

Hi Adi.

This is a good start. I was please to see a presentation that deemphasised
the action to improve the readability of the page.

The behaviour of these actions is disconcerting. When I edit a a template,
the next_url is the template, but I expected to return to the all templates
view. I know that this behaviour has not changed in this branch. I do not
see any bugs that pertain to this surprise--maybe I do not know what to
search on. Here is an example of how to negotiate multiple origins:

    @property
    def next_url(self):
        referrer = self.request.getHeader('referer')
        if (referrer is not None
            and referrer.startswith(self.request.getApplicationURL())):
            return referrer
        else:
            return canonical_url(self.context)

We should discuss if this is in scope. If not this should be a bug reported
against translations.

See below for some 3.0 style concerns.

=== modified file 'lib/lp/translations/templates/object-templates.pt'
--- lib/lp/translations/templates/object-templates.pt 2009-12-28 22:58:18 +0000
+++ lib/lp/translations/templates/object-templates.pt 2010-02-10 13:48:35 +0000
...

+ <td class="actions_column"
+ tal:condition="context/required:launchpad.AnyPerson">
+ <div class="template_links">
+ <tal:maintainer condition="template/required:launchpad.Edit">
+ <a tal:attributes="href string:${template/fmt:url}/+edit;
+ title string:Edit ${template/name}'s details">
+ <img src="/@@/edit" />&nbsp;Edit</a>
+ <a tal:attributes="href string:${template/fmt:url}/+upload;
+ title string:Upload translations to ${template/name}">
+ <img src="/@@/add" />&nbsp;Upload</a>
+ </tal:maintainer>
+ <a tal:attributes="href string:${template/fmt:url}/+export;
+ title string:Download translations from ${template/name}">
+ <img src="/@@/download" />&nbsp;Download</a>
+ <tal:admin condition="template/required:launchpad.TranslationsAdmin">
+ <a tal:attributes="href string:${template/fmt:url}/+admin;
+ title string:Administer ${template/name}">
+ <img src="/@@/edit" />&nbsp;Administer</a>
+ </tal:admin>

I an uncomfortable with these hand crafted links. We usually need extra tests
to verify that the permission and URL that is in the same as the canonical
definitions used by menus, and render the correct markup.

This rendered markup is not 3.0 style. This should use a sprite, which will
fix the strange underlining issues I see under the icon eg:

  class="sprint edit"

All these links appear to be defined in POTemplateMenu, so I think you can
avoid writing lots of tests by removing this exceptional markup with:

    <a tal:replace="structure template/menu:translations/edit/fmt:link" />

The URL is guaranteed by menu:translations. The markup is guaranteed by
fmt:link. This does permission checking for you too. This will be consistent
with other links....

review: Needs Fixing (ui)

[Adi Roiban (adiroiban)]

Hi Curtis and thanks for taking the time to do this review.

I agree that the forms next url is buggy. I think it is best to open a new bug report as this diff could become to big. If you consider this is issues is valid for a new bug,I can take care of reporting and solving that bug.

I agree that it is best to use the URL formatter, rather than constructing the URL in the template.

I think we can rename „Settings” to „Edit”.

Should we keep the javascript that hides the icons and only shows them on mouseover?

[Curtis Hovey (sinzui)]

I like the hover presentation. I want to see it on edge to get feedback from users. Please file a bug about the next URL problem.

review: Approve (ui)

[Adi Roiban (adiroiban)]

I filled bug 522188.

The hover presentation is already on edge. ie: https://translations.edge.launchpad.net/openobject-addons/trunk/+templates

I tried to look into doc/tales.txt but I am not sure if the are covering all permissions testing or if I should move the permission testing for templates actions in a different test. Any advice is much appreciated. Thanks!

Bellow is the diff with the latest changes:
=== modified file 'lib/lp/translations/browser/potemplate.py'
--- lib/lp/translations/browser/potemplate.py 2010-02-02 11:17:00 +0000
+++ lib/lp/translations/browser/potemplate.py 2010-02-15 14:55:25 +0000
@@ -175,21 +175,21 @@
     @enabled_with_permission('launchpad.Edit')
     def upload(self):
         text = 'Upload'
- return Link('+upload', text)
+ return Link('+upload', text, icon='add')

     def download(self):
         text = 'Download'
- return Link('+export', text)
+ return Link('+export', text, icon='download')

     @enabled_with_permission('launchpad.Edit')
     def edit(self):
- text = 'Settings'
- return Link('+edit', text)
+ text = 'Edit'
+ return Link('+edit', text, icon='edit')

     @enabled_with_permission('launchpad.TranslationsAdmin')
     def administer(self):
         text = 'Administer'
- return Link('+admin', text)
+ return Link('+admin', text, icon='edit')

 class POTemplateSubsetView:

=== modified file 'lib/lp/translations/templates/object-templates.pt'
--- lib/lp/translations/templates/object-templates.pt 2010-02-02 11:17:00 +0000
+++ lib/lp/translations/templates/object-templates.pt 2010-02-15 15:25:55 +0000
@@ -7,8 +7,8 @@
   <body>
     <div metal:fill-slot="head_epilogue">
       <style type="text/css">
- .inactive_links a img{
- visibility: hidden;
+ .inactive_links a{
+ background: none;
         }
         .inactive_links a{
           color: lightgray;
@@ -130,22 +130,18 @@
               <td class="actions_column"
                   tal:condition="context/required:launchpad.AnyPerson">
               <div class="template_links">
- <tal:maintainer condition="template/required:launchpad.Edit">
- <a tal:attributes="href string:${template/fmt:url}/+edit;
- title string:Edit ${template/name}'s details">
- <img src="/@@/edit" />&nbsp;Edit</a>
- <a tal:attributes="href string:${template/fmt:url}/+upload;
- title string:Upload translations to ${template/name}">
- <img src="/@@/add" />&nbsp;Upload</a>
- </tal:maintainer>
- <a tal:attributes="href string:${template/fmt:url}/+export;
- title string:Download translations from ${template/name}">
- <img src="/@@/download" />&nbsp;Download</a>
- <tal:admin condition="template/required:launchpad.TranslationsAdmin">
- <a tal:attributes="href string:${template/fmt:url}/+admin;
- title string:Administer ${template/name}">
- <img src="/@@/edit" />&nbsp;Administer...

[Curtis Hovey (sinzui)]

Thanks for these changes. These links finally meet launchpad 2.0 rules after 18 months.

We do not need explict tests for most links because the decorator and formatter rules are well tested.

[Graham Binns (gmb)]

Hi Adi,

You've got a typo in xx-series-templates.txt (line 288 of the diff):
s/lenght/length/. Other than that this is a nice branch and I'm happy for it to land. I'll run it through ec2 for you.

review: Approve

[Graham Binns (gmb)]

(Approved revision: 10102).

[Adi Roiban (adiroiban)]

Many thanks for the review!

I have pushed the typo fix.
When you have time, please run it through ec2. There is no hurry.

=== modified file 'lib/lp/translations/browser/distroseries.py'
--- lib/lp/translations/browser/distroseries.py	2009-12-27 20:58:33 +0000
+++ lib/lp/translations/browser/distroseries.py	2010-02-16 13:30:44 +0000
@@ -28,9 +28,9 @@
 
 from lp.registry.interfaces.distroseries import IDistroSeries
 from lp.translations.browser.translations import TranslationsMixin
+from lp.translations.browser.potemplate import BaseSeriesTemplatesView
 from lp.translations.interfaces.distroserieslanguage import (
     IDistroSeriesLanguageSet)
-from lp.translations.interfaces.potemplate import IPOTemplateSet
 
 
 class DistroSeriesTranslationsAdminView(LaunchpadEditFormView):
@@ -157,17 +157,12 @@
             [canonical_url(self.context), '+language-packs'])
 
 
-class DistroSeriesTemplatesView(LaunchpadView):
+class DistroSeriesTemplatesView(BaseSeriesTemplatesView):
     """Show a list of all templates for the DistroSeries."""
 
-    is_distroseries = True
-    label = "Translation templates"
-    page_title = "All templates"
-
-    def iter_templates(self):
-        potemplateset = getUtility(IPOTemplateSet)
-        return potemplateset.getSubset(distroseries=self.context,
-                                       ordered_by_names=True)
+    def initialize(self):
+        super(DistroSeriesTemplatesView, self).initialize(
+            series=self.context, is_distroseries=True)
 
 
 class DistroSeriesView(LaunchpadView, TranslationsMixin):

=== modified file 'lib/lp/translations/browser/potemplate.py'
--- lib/lp/translations/browser/potemplate.py	2010-02-01 19:19:32 +0000
+++ lib/lp/translations/browser/potemplate.py	2010-02-16 13:30:44 +0000
@@ -21,6 +21,7 @@
     'POTemplateUploadView',
     'POTemplateView',
     'POTemplateViewPreferred',
+    'BaseSeriesTemplatesView',
     ]
 
 import cgi
@@ -174,21 +175,21 @@
     @enabled_with_permission('launchpad.Edit')
     def upload(self):
         text = 'Upload'
-        return Link('+upload', text)
+        return Link('+upload', text, icon='add')
 
     def download(self):
         text = 'Download'
-        return Link('+export', text)
+        return Link('+export', text, icon='download')
 
     @enabled_with_permission('launchpad.Edit')
     def edit(self):
-        text = 'Settings'
-        return Link('+edit', text)
+        text = 'Edit'
+        return Link('+edit', text, icon='edit')
 
     @enabled_with_permission('launchpad.TranslationsAdmin')
     def administer(self):
         text = 'Administer'
-        return Link('+admin', text)
+        return Link('+admin', text, icon='edit')
 
 
 class POTemplateSubsetView:
@@ -781,3 +782,40 @@
     @property
     def text(self):
         return smartquote('Template "%s"' % self.context.name)
+
+
+class BaseSeriesTemplatesView(LaunchpadView):
+    """Show a list of all templates for the Series."""
+
+    is_distroseries = True
+    distroseries = None
+    productseries = None
+    label = "Translation templates"
+    page_title = "All templates"
+
+    def initialize(self, series, is_distroseries=True):
+        self.is_distroseries = is_distroseries
+        if is_distroseries:
+            self.distroseries = series
+        else:
+            self.productseries = series
+
+    def iter_templates(self):
+        potemplateset = getUtility(IPOTemplateSet)
+        return potemplateset.getSubset(
+            productseries=self.productseries,
+            distroseries=self.distroseries,
+            ordered_by_names=True)
+
+    def rowCSSClass(self, template):
+        if template.iscurrent:
+            return "active-template"
+        else:
+            return "inactive-template"
+
+    def isVisible(self, template):
+        if (template.iscurrent or
+            check_permission('launchpad.Edit', template)):
+            return True
+        else:
+            return False

=== modified file 'lib/lp/translations/browser/productseries.py'
--- lib/lp/translations/browser/productseries.py	2009-12-16 15:21:36 +0000
+++ lib/lp/translations/browser/productseries.py	2010-02-16 13:30:44 +0000
@@ -42,8 +42,8 @@
 
 from lp.translations.browser.poexportrequest import BaseExportView
 from lp.translations.browser.translations import TranslationsMixin
+from lp.translations.browser.potemplate import BaseSeriesTemplatesView
 from lp.code.interfaces.branchjob import IRosettaUploadJobSource
-from lp.translations.interfaces.potemplate import IPOTemplateSet
 from lp.translations.interfaces.productserieslanguage import (
     IProductSeriesLanguageSet)
 from lp.translations.interfaces.translations import (
@@ -479,18 +479,12 @@
                 _("The import has been requested."))
 
 
-class ProductSeriesTemplatesView(LaunchpadView):
+class ProductSeriesTemplatesView(BaseSeriesTemplatesView):
     """Show a list of all templates for the ProductSeries."""
 
-    is_distroseries = False
-    label = "Translation templates"
-    page_title = "All templates"
-
-    def iter_templates(self):
-        """Return an iterator of all `IPOTemplates` for the series."""
-        potemplateset = getUtility(IPOTemplateSet)
-        return potemplateset.getSubset(productseries=self.context,
-                                       ordered_by_names=True)
+    def initialize(self):
+        super(ProductSeriesTemplatesView, self).initialize(
+            series=self.context, is_distroseries=False)
 
 
 class LinkTranslationsBranchView(LaunchpadEditFormView):

=== removed file 'lib/lp/translations/stories/productseries/xx-productseries-templates.txt'
--- lib/lp/translations/stories/productseries/xx-productseries-templates.txt	2009-12-28 22:58:18 +0000
+++ lib/lp/translations/stories/productseries/xx-productseries-templates.txt	1970-01-01 00:00:00 +0000
@@ -1,95 +0,0 @@
-Templates view for ProductSeries
-================================
-
-The +templates view for ProductSeries gives an overview of the translation
-templates in this series and provides easy access to the various subpages of
-each template.
-
-
-Preparation
------------
-
-To test the ordering of templates in the listing, we need another
-template that is new but must appear at the top of the list.
-
-    >>> login('[e-mail]')
-    >>> from zope.component import getUtility
-    >>> from lp.registry.interfaces.product import IProductSet
-    >>> evolution = getUtility(IProductSet).getByName('evolution')
-    >>> evolution_trunk = evolution.getSeries('trunk')
-    >>> template = factory.makePOTemplate(productseries=evolution_trunk,
-    ...                                   name='at-the-top')
-    >>> template = factory.makePOTemplate(productseries=evolution_trunk,
-    ...                                   name='disabled')
-    >>> template.iscurrent = False
-    >>> logout()
-    >>> owner_browser = setupBrowser('Basic [e-mail]:test')
-
-Getting there
--------------
-
-To get to the listing of all templates, one needs to use the link
-from the product series translations page.
-
-    >>> user_browser.open(
-    ...     'http://translations.launchpad.dev/evolution/trunk')
-    >>> user_browser.getLink('full list of templates').click()
-    >>> print user_browser.url
-    http://translations.launchpad.dev/evolution/trunk/+templates
-
-
-The templates table
--------------------
-
-The page shows a table of all templates and links to their subpages.
-Users will not see disabled templates.
-
-    >>> table = find_tag_by_id(user_browser.contents, 'templates_table')
-    >>> print extract_text(table)
-    Template name       Last update  Actions
-    at-the-top          ...          Download
-    evolution-2.2       2005-08-25   Download
-    evolution-2.2-test  2006-12-13   Download
-
-If an administrator views this page, links to the templates admin page are
-shown, too.
-
-    >>> admin_browser.open(
-    ...     'http://translations.launchpad.dev/evolution/trunk/+templates')
-    >>> table = find_tag_by_id(admin_browser.contents, 'templates_table')
-    >>> print extract_text(table)
-    Template name       Last update  Actions
-    at-the-top          ...          Edit Upload Download Administer
-    disabled (inactive) ...          Edit Upload Download Administer
-    evolution-2.2       2005-08-25   Edit Upload Download Administer
-    evolution-2.2-test  2006-12-13   Edit Upload Download Administer
-
-
-Links to the templates
-----------------------
-
-Clicking on a template name will take the user to that template's overview
-page.
-
-    >>> admin_browser.getLink('evolution-2.2').click()
-    >>> print admin_browser.url
-    http://translations.launchpad.dev/evolution/trunk/+pots/evolution-2.2
-
-Clicking on 'Admin' will take the user to the page to administer the template.
-Likewise for the other links for each template.
-
-    >>> admin_browser.open(
-    ...     'http://translations.launchpad.dev/evolution/trunk/+templates')
-    >>> admin_browser.getLink('Admin', index=0).click()
-    >>> print admin_browser.url
-    http://translations.launchpad.dev/evolution/trunk/+pots/at-the-top/+admin
-
-Clicking on 'Edit' will take the user to the page to edit the template
-details, even if the translation of the templates are not active.
-
-    >>> owner_browser.open(
-    ...     'http://translations.launchpad.dev/evolution/trunk/+templates')
-    >>> owner_browser.getLink('Edit', index=1).click()
-    >>> print owner_browser.url
-    http://translations.launchpad.dev/evolution/trunk/+pots/disabled/+edit
-

=== renamed file 'lib/lp/translations/stories/distroseries/xx-distroseries-templates.txt' => 'lib/lp/translations/stories/standalone/xx-series-templates.txt'
--- lib/lp/translations/stories/distroseries/xx-distroseries-templates.txt	2009-12-10 12:46:11 +0000
+++ lib/lp/translations/stories/standalone/xx-series-templates.txt	2010-02-16 13:30:44 +0000
@@ -1,5 +1,9 @@
-Templates view for DistroSeries
-===============================
+Templates view for Series
+=========================
+
+
+Templates for DistroSeries
+--------------------------
 
 The +templates view for DistroSeries gives an overview of the translation
 templates in this series and provides easy access to the various subpages of
@@ -19,8 +23,8 @@
     http://translations.launchpad.dev/ubuntu/hoary/+templates
 
 
-The templates table
--------------------
+Templates view for DistroSeries
+-------------------------------
 
 Full template listing for a distribution series is reached by following
 a link from the distribution series translations page.
@@ -29,18 +33,14 @@
     ...     'http://translations.launchpad.dev/ubuntu/hoary')
     >>> anon_browser.getLink('full list of templates').click()
 
-Full listing of templates shows source package name, template name and
-the date of last update for this distribution series.
+Full listing of templates shows priority, source package name, template name
+length, languages and the date of last update for this distribution series.
 
     >>> table = find_tag_by_id(anon_browser.contents, 'templates_table')
     >>> print extract_text(table)
-    Source package Template name      Last update
-    evolution      evolution-2.2      2005-05-06
-    evolution      man                2006-08-14
-    mozilla        pkgconf-mozilla    2005-05-06
-    pmount         man                2006-08-14
-    pmount         pmount             2005-05-06
-
+    Priority    Source package Template name    Length  Languages   Updated
+    100         evolution      evolution-2.2    22      2           2005-05-06
+    0           evolution      man              1       1           2006-08-14 ...
 
 Logged-in users see a link to all the active translation templates
 on a distribution series translation page.
@@ -53,11 +53,9 @@
 
     >>> table = find_tag_by_id(user_browser.contents, 'templates_table')
     >>> print extract_text(table)
-    Source package Template name     Last update  Actions
-    evolution      evolution-2.2     2005-05-06   Download
-    ...
-    mozilla        pkgconf-mozilla   2005-05-06   Download
-    ...
+    Priority    Source package  Template name ...   Updated     Actions
+    100         evolution       evolution-2.2 ...   2005-05-06  Download ...
+    0           mozilla         pkgconf-mozilla ... 2005-05-06  Download ...
 
 Administrator can see all editing options.
 
@@ -69,35 +67,13 @@
 
     >>> table = find_tag_by_id(admin_browser.contents, 'templates_table')
     >>> print extract_text(table)
-    Source package Template name                Last update  Actions
-    evolution      disabled-template (inactive) 2007-01-05   Edit Upload Download Administer
-    evolution      evolution-2.2                2005-05-06   Edit Upload Download Administer
-    evolution      man                          2006-08-14   Edit Upload Download Administer
-    mozilla        pkgconf-mozilla              2005-05-06   Edit Upload Download Administer
-    pmount         man                          2006-08-14   Edit Upload Download Administer
-    pmount         pmount                       2005-05-06   Edit Upload Download Administer
-
-Ubuntu Translations Coordinator can administer all templates, including
-those that are currently disabled.
-
-New user is made an owner of an Ubuntu translation group, thus being
-considered a translations administrator for Ubuntu.
-
-    >>> from zope.component import getUtility
-    >>> from canonical.launchpad.ftests import login, logout
-    >>> from canonical.launchpad.interfaces import (
-    ...     IDistributionSet, IPersonSet, ILaunchpadCelebrities)
-    >>> login('[e-mail]')
-    >>> a_person = factory.makePerson(email='[e-mail]',
-    ...     name='utc-member', password='test', displayname='Some Guy')
-    >>> utc_member = factory.makeTranslator('ro', person=a_person)
-    >>> utc_team = factory.makeTeam(owner=a_person)
-    >>> utg = factory.makeTranslationGroup(owner=utc_team,
-    ...                                 name="Ubuntu Translation Group")
-    >>> ubuntu = getUtility(ILaunchpadCelebrities).ubuntu
-    >>> ubuntu.translationgroup = utg
-    >>> # Log out so we can go back to using test browsers.
-    >>> logout()
+    Priority    Source package  ... Updated     Actions
+    0           evolution       ... 2007-01-05  Edit Upload Download Administer
+    100 ...
+    0           pmount          ... 2005-05-06  Edit Upload Download Administer
+
+Translation group owners can administer all templates for their distribution,
+including those that are currently disabled.
 
 Visiting the Ubuntu Hoary templates page, translation administrators
 will see "Edit" and "Administer" links for all templates.
@@ -105,7 +81,7 @@
 Trying to edit/administer enabled templates brings them to the
 appropriate page.
 
-    >>> utc_browser = setupBrowser(auth='Basic [e-mail]:test')
+    >>> utc_browser = setupDTCBrowser()
     >>> utc_browser.open(
     ...     'http://translations.launchpad.dev/ubuntu/hoary/+templates')
     >>> utc_browser.getLink(url='ubuntu/hoary/+source/evolution/+pots/evolution-2.2/+edit').click()
@@ -121,7 +97,6 @@
 Trying to edit/administer disabled templates brings them to the
 appropriate page.
 
-    >>> utc_browser = setupBrowser(auth='Basic [e-mail]:test')
     >>> utc_browser.open(
     ...     'http://translations.launchpad.dev/ubuntu/hoary/+templates')
     >>> utc_browser.getLink(url='ubuntu/hoary/+source/evolution/+pots/disabled-template/+edit').click()
@@ -154,3 +129,84 @@
     >>> print admin_browser.url
     http://translations.../evolution/+pots/disabled-template/+edit
 
+
+Templates view for ProductSeries
+--------------------------------
+
+The +templates view for ProductSeries gives an overview of the translation
+templates in this series and provides easy access to the various subpages of
+each template.
+
+
+Preparation
+-----------
+
+To test the ordering of templates in the listing, we need another
+template that is new but must appear at the top of the list.
+
+    >>> login('[e-mail]')
+    >>> from zope.component import getUtility
+    >>> from lp.registry.interfaces.product import IProductSet
+    >>> evolution = getUtility(IProductSet).getByName('evolution')
+    >>> evolution_trunk = evolution.getSeries('trunk')
+    >>> template = factory.makePOTemplate(productseries=evolution_trunk,
+    ...                                   name='at-the-top')
+    >>> logout()
+
+
+Getting there
+-------------
+
+To get to the listing of all templates, one needs to use the link
+from the product series translations page.
+
+    >>> user_browser.open(
+    ...     'http://translations.launchpad.dev/evolution/trunk')
+    >>> user_browser.getLink('full list of templates').click()
+    >>> print user_browser.url
+    http://translations.launchpad.dev/evolution/trunk/+templates
+
+
+The templates table
+-------------------
+
+The page shows a table of all templates and links to their subpages.
+
+    >>> table = find_tag_by_id(user_browser.contents, 'templates_table')
+    >>> print extract_text(table)
+    Priority    Template name   Length  Languages   Updated     Actions
+    0           at-the-top      0       0           ...         Download
+    0           evolution-2.2   22      1           2005-08-25  Download
+    0           evolutio...     8       1           2006-12-13  Download
+
+If an administrator views this page, links to the templates admin page are
+shown, too.
+
+    >>> admin_browser.open(
+    ...     'http://translations.launchpad.dev/evolution/trunk/+templates')
+    >>> table = find_tag_by_id(admin_browser.contents, 'templates_table')
+    >>> print extract_text(table)
+    Priority    Template name ...   Updated     Actions
+    0           at-the-top    ...   ...         Edit Upload Download Administer
+    0           evolution-2.2 ...   2005-08-25  Edit Upload Download Administer
+    0           evolutio...   ...   2006-12-13  Edit Upload Download Administer
+
+
+Links to the templates
+----------------------
+
+Clicking on a template name will take the user to that template's overview
+page.
+
+    >>> admin_browser.getLink('evolution-2.2').click()
+    >>> print admin_browser.url
+    http://translations.launchpad.dev/evolution/trunk/+pots/evolution-2.2
+
+Clicking on 'Edit' will take the user to the page to edit the template
+details. Likewise for the other links for each template.
+
+    >>> admin_browser.open(
+    ...     'http://translations.launchpad.dev/evolution/trunk/+templates')
+    >>> admin_browser.getLink('Edit').click()
+    >>> print admin_browser.url
+    http://translations.launchpad.dev/evolution/trunk/+pots/at-the-top/+edit

=== modified file 'lib/lp/translations/templates/object-templates.pt'
--- lib/lp/translations/templates/object-templates.pt	2009-12-28 22:58:18 +0000
+++ lib/lp/translations/templates/object-templates.pt	2010-02-16 13:30:44 +0000
@@ -7,8 +7,8 @@
   <body>
     <div metal:fill-slot="head_epilogue">
       <style type="text/css">
-        .inactive_links a img{
-          visibility: hidden;
+        .inactive_links a{
+          background: none;
         }
         .inactive_links a{
           color: lightgray;
@@ -16,9 +16,11 @@
         .template_links a{
           padding-right: 1em;
         }
-        td.template_column, th.template_column,
         td.sourcepackage_column, th.sourcepackage_column {
-            width: 20em;
+            width: 10em;
+        }
+        td.template_column, th.template_column {
+            width: 14em;
         }
         td.actions_column, th.actions_column {
             width: 27em;
@@ -26,14 +28,14 @@
       </style>
       <style tal:condition="view/is_distroseries" type="text/css">
         #templates_table {
+            width: 81em;
+        }
+      </style>
+      <style tal:condition="not:view/is_distroseries" type="text/css">
+        #templates_table {
             width: 79em;
         }
       </style>
-      <style tal:condition="not:view/is_distroseries" type="text/css">
-        #templates_table {
-            width: 58em;
-        }
-      </style>
       <script language="JavaScript" type="text/javascript">
       LPS.use('node-base', 'event-delegate', function(Y) {
           Y.on('domready', function(e) {
@@ -72,10 +74,13 @@
       <table class="sortable listing" id="templates_table">
         <thead>
           <tr>
+            <th class="priority_column">Priority</th>
             <th tal:condition="view/is_distroseries"
                 class="sourcepackage_column">Source package</th>
             <th class="template_column">Template name</th>
-            <th class="lastupdate_column">Last update</th>
+            <th class="length_column">Length</th>
+            <th class="languages_column">Languages</th>
+            <th class="lastupdate_column">Updated</th>
             <th class="actions_column"
                 tal:condition="context/required:launchpad.AnyPerson">
               Actions</th>
@@ -83,105 +88,63 @@
         </thead>
         <tbody>
           <tal:templates repeat="template view/iter_templates">
-            <tal:not-current condition="not: template/iscurrent">
-              <tal:admin condition="template/required:launchpad.Edit">
-                <tr class="template_row inactive-template">
-                  <td tal:condition="view/is_distroseries"
-                      tal:content="template/sourcepackagename/name"
-                      class="sourcepackage_column">Source package
-                  </td>
-                  <td class="template_column">
-                    <a tal:attributes="href template/fmt:url">
-                      <span tal:content="template/name">Template name</span>
-                    </a> (inactive)
-                  </td>
-                  <td class="lastupdate_column">
-                    <span class="sortkey"
-                          tal:condition="template/date_last_updated"
-                          tal:content="template/date_last_updated/fmt:datetime">
-                          time sort key
-                    </span>
-                    <span class="lastupdate_column"
-                          tal:condition="template/date_last_updated"
-                          tal:attributes="
-                            title template/date_last_updated/fmt:datetime"
-                          tal:content="
-                            template/date_last_updated/fmt:approximatedate"
-                    >
-                       2009-09-23
-                    </span>
-                  </td>
-                  <td class="actions_column"
-                      tal:condition="context/required:launchpad.AnyPerson">
-                  <div class="template_links">
-                    <tal:maintainer condition="template/required:launchpad.Edit">
-                      <a tal:attributes="href string:${template/fmt:url}/+edit;
-                                         title string:Edit ${template/name}'s details">
-                         <img src="/@@/edit" />&nbsp;Edit</a>
-                      <a tal:attributes="href string:${template/fmt:url}/+upload;
-                            title string:Upload translations to ${template/name}">
-                         <img src="/@@/add" />&nbsp;Upload</a>
-                    </tal:maintainer>
-                    <a tal:attributes="href string:${template/fmt:url}/+export;
-                         title string:Download translations from ${template/name}">
-                      <img src="/@@/download" />&nbsp;Download</a>
-                    <tal:admin condition="template/required:launchpad.TranslationsAdmin">
-                      <a tal:attributes="href string:${template/fmt:url}/+admin;
-                                         title string:Administer ${template/name}">
-                         <img src="/@@/edit" />&nbsp;Administer</a>
-                    </tal:admin>
-                  </div>
-                  </td>
-                </tr>
-              </tal:admin>
-            </tal:not-current>
-            <tal:current condition="template/iscurrent">
-              <tr class="template_row">
-                <td tal:condition="view/is_distroseries"
-                    tal:content="template/sourcepackagename/name"
-                    class="sourcepackage_column">Source package
-                </td>
-                <td class="template_column"><a tal:attributes="href template/fmt:url"
-                       tal:content="template/name">Template name</a></td>
-                <td class="lastupdate_column">
-                  <span class="sortkey"
-                        tal:condition="template/date_last_updated"
-                        tal:content="template/date_last_updated/fmt:datetime">
-                        time sort key
-                  </span>
-                  <span class="lastupdate_column"
-                        tal:condition="template/date_last_updated"
-                        tal:attributes="
-                          title template/date_last_updated/fmt:datetime"
-                        tal:content="
-                          template/date_last_updated/fmt:approximatedate"
-                  >
-                     2009-09-23
-                  </span>
-                </td>
-                <td class="actions_column"
-                    tal:condition="context/required:launchpad.AnyPerson">
-                <div class="template_links">
-                  <tal:maintainer condition="template/required:launchpad.Edit">
-                    <a tal:attributes="href string:${template/fmt:url}/+edit;
-                                       title string:Edit ${template/name}'s details">
-                       <img src="/@@/edit" />&nbsp;Edit</a>
-                    <a tal:attributes="href string:${template/fmt:url}/+upload;
-                          title string:Upload translations to ${template/name}">
-                       <img src="/@@/add" />&nbsp;Upload</a>
-                  </tal:maintainer>
-                  <a tal:attributes="href string:${template/fmt:url}/+export;
-                       title string:Download translations from ${template/name}">
-                    <img src="/@@/download" />&nbsp;Download</a>
-                  <tal:admin condition="template/required:launchpad.TranslationsAdmin">
-                    <a tal:attributes="href string:${template/fmt:url}/+admin;
-                                       title string:Administer ${template/name}">
-                       <img src="/@@/edit" />&nbsp;Administer</a>
-                  </tal:admin>
-                </div>
-                </td>
-              </tr>
-            </tal:current>
+            <tr tal:define="
+                  inactive_css_class python:view.rowCSSClass(template)"
+                tal:condition="python: view.isVisible(template)"
+                tal:attributes="
+                  class string: template_row ${inactive_css_class}">
+              <td class="priority_column"
+                  tal:content="template/priority">7</td>
+              <td tal:condition="view/is_distroseries"
+                  tal:content="template/sourcepackagename/name"
+                  class="sourcepackage_column">Source package
+              </td>
+              <td class="template_column">
+                <a tal:attributes="href template/fmt:url">
+                  <span tal:content="template/name">Template name</span>
+                </a>
+                <tal:inactive condition="not: template/iscurrent">
+                  (inactive)
+                </tal:inactive>
+              </td>
+              <td class="length_column"
+                  tal:content="template/getPOTMsgSetsCount">1777</td>
+              <td class="languages_column"
+                  tal:content="template/language_count">777</td>
+              <td class="lastupdate_column">
+                <span class="sortkey"
+                      tal:condition="template/date_last_updated"
+                      tal:content="template/date_last_updated/fmt:datetime">
+                      time sort key
+                </span>
+                <span class="lastupdate_column"
+                      tal:condition="template/date_last_updated"
+                      tal:attributes="
+                        title template/date_last_updated/fmt:datetime"
+                      tal:content="
+                        template/date_last_updated/fmt:approximatedate"
+                >
+                   2009-09-23
+                </span>
+              </td>
+              <td class="actions_column"
+                  tal:condition="context/required:launchpad.AnyPerson">
+              <div class="template_links">
+                <a tal:replace="
+                    structure template/menu:translations/edit/fmt:link
+                    " />
+                <a tal:replace="
+                    structure template/menu:translations/upload/fmt:link
+                    " />
+                <a tal:replace="
+                    structure template/menu:translations/download/fmt:link
+                    " />
+                <a tal:replace="
+                    structure template/menu:translations/administer/fmt:link
+                    " />
+              </div>
+              </td>
+            </tr>
           </tal:templates>
         </tbody>
       </table>