Closed Bug 571143 Opened 14 years ago Closed 14 years ago

Support Windows 64 bit builds in TinderboxPushlog

Categories

(Tree Management Graveyard :: TBPL, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file)

Attached patch Patch (v1)Splinter Review
I also tweaked the matrix styling a bit so that more stuff fits on the screen.
Attachment #450260 - Flags: review?(mstange)
Summary: Support Windows 64 bit builds → Support Windows 64 bit builds in TinderboxPushlog
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
See <http://ehsanakhgari.org/tinderboxpushlog/index.html> if you need an example of how things with my patch look like.
Comment on attachment 450260 [details] [diff] [review]
Patch (v1)

Thanks for picking up my slack!
Just one thing: Can you increase the width of ".results .os" to 65 or 70px (and likewise the margin-left of .osresults)? Not sure how this will look on different platforms, but on Mac it's almost overlapping, so I think some pixels more are in order.
Attachment #450260 - Flags: review?(mstange) → review+
This does make "Windows 64" overlap the results section too for me. Perhaps just "Win64" instead for brevity rather than increasing space used?
Yeah, maybe that's better. I think I did the same for "OSX 64".
Any chance this can land soon? The Win64 builds are ending up in the 'Win' row, which is a bit confusing.
http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/449302485960

(In reply to comment #4)
> This does make "Windows 64" overlap the results section too for me. Perhaps
> just "Win64" instead for brevity rather than increasing space used?

I decided to do that, because Tinderbox is down right now, and I couldn't test the proposed CSS change.

(In reply to comment #2)
> Just one thing: Can you increase the width of ".results .os" to 65 or 70px (and
> likewise the margin-left of .osresults)? Not sure how this will look on
> different platforms, but on Mac it's almost overlapping, so I think some pixels
> more are in order.

Hmm, I think a better way to fix the overlap could be changing the width of matrix columns.  However, I'd personally prefer there to be a deviding line between them and having them as close to each other as possible.  That would make the best use of space, without causing confusion.  Right?

(In reply to comment #5)
> Yeah, maybe that's better. I think I did the same for "OSX 64".

Nope, you didn't.  :-)  I was about to do that when landing, but maybe that's not useful now, because we do get talos results for OSX64, and therefore the width is really used.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
(In reply to comment #7)
> http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/449302485960

Deployed.

> (In reply to comment #4)
> Hmm, I think a better way to fix the overlap could be changing the width of
> matrix columns.

Hmm? I was thinking of the labels in front of the results in the push list, not of the matrix.
Ah, then I totally didn't understand what you meant the first time!

http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/1aec6b70a744
Blocks: support-L64
No longer blocks: support-L64
Probably a good idea to hold off on deploying this until bug 570882 is fixed, since the Win64 builds are permared and hidden right now.
Depends on: 570882
It won't hurt to land it but we won't be able to verify it until I unhide the builds.
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: