486942 - Download page update: localized templates

Status: RESOLVED FIXED

(Bugzilla :: bugzilla.org, enhancement)

Description

[Vitaly Fedrushkov]

Attachment: Patch

Layout optimized once more

Available versions updated after recent release

Comment 1

[Max Kanat-Alexander]

Comment on attachment 371138 [details] [diff] [review]
Patch

>--- bugzilla-org/src/download/index.html
>+  <tr><th>[% release.legacy.branch %]</th>
>+      <th>[% release.old.branch %]</th>
>+      <th>[% release.stable.branch %]</th>
>+      <th>[% release.dev.branch %]</th>
>+  </tr>

  Just FYI, I don't think it's a good idea to put those in as variables, as they will change when we have a new major stable release, but I will not necessarily fix the localizations section.

Comment 2

[Frédéric Buclin]

Yeah, it's probably better to hardcode branch names. Vitaly, do you have a screenshot of the updated page?

Comment 3

[Vitaly Fedrushkov]

Attachment: Patch v2

(In reply to comment #1)
>   Just FYI, I don't think it's a good idea to put those in as variables, as
> they will change when we have a new major stable release, but I will not
> necessarily fix the localizations section.

Valid point, fixed.

Also, page in Doctor is somewhat wider, so there's a need for 'width' or more cellspacing to keep numbers from crowding.

Comment 4

[Vitaly Fedrushkov]

Attachment: Screenshot of v2

Here's how it looks now

Comment 5

[Frédéric Buclin]

Comment on attachment 372169 [details] [diff] [review]
Patch v2

The UI looks good, and I see nothing wrong with the code. r=LpSolit

Comment 6

[Frédéric Buclin]

Checking in bugzilla-org/src/download/index.html;
/www/bugzilla-org/src/download/index.html,v  <--  index.html
new revision: 1.140; previous revision: 1.139
done