Closed Bug 873884 Opened 11 years ago Closed 11 years ago

Update trychooser website

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: philor, Assigned: philor)

Details

(Keywords: trychooser)

Attachments

(2 files)

Platforms:

* otoro doesn't exist
* b2g emulator does exist

Unittests:

* android doesn't have crashtest-2 and crashtest-3, only crashtest
* android does have mochitest-gl
* b2g has reftest-7 through reftest-10
* b2g has crashtest-1 through crashtest-3

Talos:

* desktop is in the same order tbpl displays, so it's easy to compare; android might almost but not quite be in length-of-name order
* android remote-tdhtml and remote-tpan don't exist
I'm not a stickler for validation, but it does come in handy sometimes - if you look at http://validator.nu/?doc=http%3A%2F%2Ftrychooser.pub.build.mozilla.org%2F&parser=html5&showsource=yes you'll see a crapload of errors, some of which don't matter at all, a few of which are maybe better than the alternatives, and one of which, hidden away amongst all the others, is the reason the queue for Android-x86 doesn't work.

Every time I think about touching this page, I start by doing this, and generally then just wander off. Maybe if I land it, I'll stop wandering off.

The only real visible change is that splitting the <ol>s into separate lists instead of lists which suddenly have an invalid <h4> in them makes things like "Android-only Unittest Suites" span the whole column instead of being indented. Don't care. The only remaining genuine error is the div wrapped around the mochitest option-subgroup, which someone who really wanted to fix could fix. Don't want to.
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #754305 - Flags: review?(emorley)
Attached patch UpdateSplinter Review
Per comment 0.
Attachment #754306 - Flags: review?(emorley)
Attachment #754305 - Flags: review?(emorley) → review+
Comment on attachment 754306 [details] [diff] [review]
Update

Looks good & works fine locally - thank you for doing this :-)
Attachment #754306 - Flags: review?(emorley) → review+
Deployed with bug 877465.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
Component: Tools → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: