Closed Bug 1566358 Opened 6 years ago Closed 5 years ago

Update releases page to include 64-bit Windows and Linux platforms

Categories

(SeaMonkey :: Website, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

Details

Attachments

(23 files, 3 obsolete files)

56.24 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
6.37 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
5.01 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
3.99 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
877 bytes, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
29.15 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
86.23 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
1.70 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
854 bytes, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
1.49 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
792 bytes, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
822 bytes, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
59.98 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
3.65 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
2.73 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
5.13 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
26.72 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
40.34 KB, patch
frg
: review+
Details | Diff | Splinter Review
13.79 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
24.07 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
11.85 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
25.95 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review
20.63 KB, patch
iannbugzilla
: review+
Details | Diff | Splinter Review

At the moment release pages only cater for multiple locales on 32-bit Windows and Linux and this needs to be extended to include 64-bit Windows and Linux.

Attachment #9078346 - Attachment description: New files to test changes → New files to test changes [Checked in: Comment 2]
Attachment #9078346 - Flags: review+
Attached patch Fix typos and coding (obsolete) — Splinter Review
Attachment #9078414 - Flags: review+
Attached patch Fix typos and coding v2 (obsolete) — Splinter Review
Attachment #9078414 - Attachment is obsolete: true
Attachment #9078422 - Flags: review+

Set class on download link too

Attachment #9078422 - Attachment is obsolete: true
Attachment #9078429 - Flags: review+
Attachment #9078429 - Attachment description: Fix typos and coding v3 → Fix typos and coding v3 [Checked in: Comment 6]
Attachment #9078477 - Attachment description: Styling for new index → Styling for new index [Checked in: Comment 8]
Depends on: 1566647
Attachment #9079612 - Attachment description: Fix divs and add more styling → Fix divs and add more styling [Checked in: Comment 10]
Attachment #9079613 - Attachment description: Fix width of download box → Fix width of download box [Checked in: Comment 12]

Lots of duplication between this and the releases page (though mostly hidden).

Attachment #9079614 - Flags: review+
Attachment #9079614 - Attachment description: Updated download section of home page → Updated download section of home page [Checked in: Comment 14]

There is no point having the same generic title on every entry in the table so remove them and use a single attribute elsewhere.

Attachment #9079629 - Flags: review+
Attachment #9079629 - Attachment description: Remove duplication in titles → Remove duplication in titles [Checked in: Comment 16]

Table footers need to take account of 8 rather than 6 columns.

Attachment #9079630 - Flags: review+
Attachment #9079630 - Attachment description: Fix table footers → Fix table footers [Checked in: Comment 18]

As ewong pointed out, was missing a couple of variables so not being processed properly.

Attachment #9079683 - Flags: review+
Attachment #9079683 - Attachment description: Fix new home page → Fix new home page [Checked in: Comment 20]

Missed the change to JS to update download link title from data-title attribute.

Attachment #9079686 - Flags: review+
Attachment #9079686 - Attachment description: Fix title JS → Fix title JS [Checked in: Comment 22]

On the home page there was too much green space below the bullet points on the download now box.

Attachment #9079692 - Flags: review+
Attachment #9079692 - Attachment description: Fix Green Space → Fix Green Space [Checked in: Comment 24]

There's not a big enough margin below the download box on the homepage which means some of the text almost overruns the key-point box.

Attachment #9079704 - Flags: review+
Attachment #9079704 - Attachment description: Fix margin on homepage download box → Fix margin on homepage download box [Checked in: Comment 26]

Rather than repeating the same code across multiple HTML pages, make use of the templates with XML sources for the data.

Attachment #9080402 - Flags: review+
Attachment #9080402 - Attachment description: Use xml source for downloads table and fieldset → Use xml source for downloads table and fieldset [Checked in: Comment 28]

Switch to using x64/x32 instead of 64-bit/32-bit
Fix issue with key-point divs being too short.

Attachment #9080417 - Flags: review+
Attachment #9080417 - Attachment description: Switch from bit → Switch from bit [Checked in: Comment 30]

Accidentally missed out the <p> tags from the fieldset XML conversion

Attachment #9080419 - Flags: review+
Attachment #9080419 - Attachment description: Fix fieldset → Fix fieldset [Checked in: Comment 32]

Allow the table footer to be defined by XML too
Remove unneeded class from 6th column heading
Correct order of languages in table

Attachment #9080496 - Flags: review+
Attachment #9080496 - Attachment description: Table footer by XML → Table footer by XML [Checked in: Comment 34]

Moves JS into separate file so it can be shared with other start pages
Restructures divs so there is only one download box
Switches new page to use XML source for table and fieldset

Attachment #9080497 - Flags: review+
Attachment #9080497 - Attachment description: New start page → New start page [Checked in: Comment 36]
Attached patch New de start page (obsolete) — Splinter Review

New de page that:

  • Uses common JS rather than its own copy
  • Restructures divs so there is a single download box
  • Uses localised XML sources to provide table and fieldset

All localisation additions need a native speaker to review as the internet is not the best at translating strings.

In theory other de pages could be created (release / home) using the same localised XML sources.

Attachment #9080500 - Flags: review?(frgrahl)

Fixed an issue I found with CSS and a missing table column

Attachment #9080500 - Attachment is obsolete: true
Attachment #9080500 - Flags: review?(frgrahl)

Fixes for:

  • Missing column version in XML cannot be set to "" so set to "ver"
  • CSS for start page's congratulations div needs adjusting for size of SM logo
Attachment #9080504 - Flags: review+
Attachment #9080504 - Attachment description: New fixes → New fixes [Checked in: Comment 40]
Attachment #9080502 - Flags: review?(frgrahl)
Comment on attachment 9080502 [details] [diff] [review] New de start page v1.1 [Checked in: Comment 42] Review of attachment 9080502 [details] [diff] [review]: ----------------------------------------------------------------- > SeaMonkey-Builds können heruntergeladen werden. Looks good. Thinking about changing the above string later but it is correct. Just sounds a little "wooden". But can go in.
Attachment #9080502 - Flags: review?(frgrahl) → review+
Attachment #9080502 - Attachment description: New de start page v1.1 → New de start page v1.1 [Checked in: Comment 42]
Attachment #9083745 - Attachment description: Switch 2.49.4 to use xml file for table → Switch 2.49.4 to use xml file for table [Checked in: Comment 46]
Attachment #9083752 - Attachment description: Switch releases index to use XML file for table → Switch releases index to use XML file for table [Checked in: Comment 48]
Attachment #9083803 - Attachment description: Update de and en start pages → Update de and en start pages [Checked in: Comment 50]

Some very old release pages have green download boxes, these are not needed any more.

Attachment #9083824 - Flags: review+
Attachment #9083803 - Attachment description: Update de and en start pages [Checked in: Comment 50] → Update de and en start pages [Checked in: Comment 48]
Attachment #9083824 - Attachment description: Remove old green download boxes → Remove old green download boxes [Checked in: Comment 50]
Attachment #9083752 - Attachment description: Switch releases index to use XML file for table [Checked in: Comment 48] → Switch releases index to use XML file for table [Checked in: Comment 46]
Attachment #9083745 - Attachment description: Switch 2.49.4 to use xml file for table [Checked in: Comment 46] → Switch 2.49.4 to use xml file for table [Checked in: Comment 44]

SeaMonkey 2.49.5 was released and this all seems to work fine. Thanks to all involved especially to IanN for redoing the templates.

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: