536371 - Make existing localizer pages work with new layout

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Localization, defect, P3)

Description

[Fred Wenzel [:wenzel]]

The existing localizer pages need some amo2009 love.

Comment 1

[Fred Wenzel [:wenzel]]

I adapted these pages to the new layout in r58789.

QA: If the Applications, Platform, etc. pages do not look ridiculously unusable, this worked ;) One that was particularly broken in the new layout is the "Pages" page (/localizers/pages).

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED on:

* https://preview.addons.mozilla.org/en-US/localizers/pages
* https://preview.addons.mozilla.org/en-US/localizers/platforms
* https://preview.addons.mozilla.org/en-US/localizers/categories
* https://preview.addons.mozilla.org/en-US/localizers/gettext
* https://preview.addons.mozilla.org/en-US/localizers/collection_features
* https://preview.addons.mozilla.org/en-US/localizers/applications

I looked a couple RTL-language pages such as https://preview.addons.mozilla.org/he/localizers/categories, but, Ehsan, please do take a look at those too, when you get a chance.  Thanks!

Comment 3

[(no longer active)]

RTL report:

> * https://preview.addons.mozilla.org/en-US/localizers/pages

Looks fine.

> * https://preview.addons.mozilla.org/en-US/localizers/platforms

Got an access denied error.

> * https://preview.addons.mozilla.org/en-US/localizers/categories

Looks fine.

> * https://preview.addons.mozilla.org/en-US/localizers/gettext

This one's fine as well.

> * https://preview.addons.mozilla.org/en-US/localizers/collection_features

Fine as well.

> * https://preview.addons.mozilla.org/en-US/localizers/applications

Ditto.

Comment 4

[Fred Wenzel [:wenzel]]

Great, thanks Ehsan. That you got "access denied" on platforms is fine, that went away in bug 538093.

Comment 5

[(no longer active)]

(In reply to comment #4)
> Great, thanks Ehsan. That you got "access denied" on platforms is fine, that
> went away in bug 538093.

I couldn't tell if that bug is supposed to fix the "access denied" error, or if that is, in fact, expected.