718980 - [bugsy] make team snippet more constructive, now that team page is

Status: RESOLVED FIXED

(Webtools Graveyard :: Elmo, defect, P2)

Description

[Axel Hecht [:Pike]]

Now that we've done a bit jump for the team page, we should make bugs more prominent. I'll do some short experiments to gather feedback on.

Comment 1

[Axel Hecht [:Pike]]

Published the initial step at https://github.com/Pike/elmo/compare/develop...feature%2Fbug-718980-bugsy-team-snippet.

Changes: I reworded the heading and the intro blurb to add a tad of context. And I show the ID, summary, last change date and creation date for the most recent 5 open bugs.

I didn't do any HTML styling on that yet. Mostly because it's hard to tell how things look once the design lands, and because I would like to re-use the styles for the tables from the shipping snippet. But for that, we'd had to rename/add classes there, so I wanted feedback on that from peterbe.

So, peterbe, should we make the styling more generic for the tables?

And Jeff, comments on the copy here?

Comment 2

[Peter Bengtsson [:peterbe]]

I like it! There are a couple of things I'd like to improve/change so please go ahead and make a patch and I'll try it out and review.

Comment 3

[Jeff Beatty [:gueroJeff]]

(In reply to Axel Hecht [:Pike] from comment #1)
> Published the initial step at
> https://github.com/Pike/elmo/compare/develop...feature%2Fbug-718980-bugsy-
> team-snippet.
> 
> Changes: I reworded the heading and the intro blurb to add a tad of context.
> And I show the ID, summary, last change date and creation date for the most
> recent 5 open bugs.
> 
> I didn't do any HTML styling on that yet. Mostly because it's hard to tell
> how things look once the design lands, and because I would like to re-use
> the styles for the tables from the shipping snippet. But for that, we'd had
> to rename/add classes there, so I wanted feedback on that from peterbe.
> 
> So, peterbe, should we make the styling more generic for the tables?
> 
> And Jeff, comments on the copy here?

Copy looks great to me. Short, sweet, and to the point. And yes, I did just say that something you wrote was sweet ;-)

Comment 4

[Peter Bengtsson [:peterbe]]

Attachment: BugPuller

I took your branch and revamped it a bit.
* Added "Please wait..." loading whilst waiting for the AJAX calls
* An error catcher that shows and excuse (you can run `BugPuller.excuse()` in the Web Console)
* Tidied up the code a bit

You can see my branch in action here:
https://github.com/peterbe/elmo/compare/develop...feature%2Fbug-718980-bugsy-team-snippet

Comment 5

[Axel Hecht [:Pike]]

Comment on attachment 593123 [details] [diff] [review]
BugPuller

I like what you did to the code here, r=me.

Comment 6

[Peter Bengtsson [:peterbe]]

https://github.com/mozilla/elmo/commit/32b33eb51eea873d6f71e438e42753a851633a28

Comment 7

[Peter Bengtsson [:peterbe]]

Actually some "regression". We ought to insert some timeout callbacks. Right now, when I'm testing, it's not failing simply because it's allowed to take a loooong time.

Comment 8

[Peter Bengtsson [:peterbe]]

(In reply to Peter Bengtsson [:peterbe] from comment #7)
> Actually some "regression". We ought to insert some timeout callbacks. Right
> now, when I'm testing, it's not failing simply because it's allowed to take
> a loooong time.

I take it back. It's working just fine. It's just that the bugzilla.mozilla.org server is configured for very very long read time-outs. We shouldn't be opinionated in the AJAX and just let bugzilla time out when it needs to time out.