1210690 - Display only commits and only relevant data

Status: RESOLVED FIXED

(bugzilla.mozilla.org Graveyard :: Extensions: MozReview Integration, defect)

Description

[Mark Côté [:mcote]]

There are currently two confusing factors in the MozReview table:

* Parents are displayed, and we want to discourage reviews on them.
* The submitter is only displayed as part of the cryptic parent summary.
* The status field is very Review Board specific and not useful.

Thus we should take out the parent row altogether, add a submitter column (ideally a standard BMO user widget, but at least an IRC nick), and remove the status column.

Comment 1

[Mark Côté [:mcote]]

Screenshot at https://reviewboard.mozilla.org/r/21059/, with submitter nicks in plain text.

Comment 2

[Mark Côté [:mcote]]

Attachment: Simplify MozReview table

Had to add some !important CSS attributes sadly, but saw no way around it.

As discussed, has text-only submitter nicks.  Full UI is pending templatification of the user widget.

Comment 3

[Mark Côté [:mcote]]

Heh I guess that's three confusing factors, in the end.

Comment 4

[:glob ✱]

Comment on attachment 8668815 [details] [diff] [review]
Simplify MozReview table

Review of attachment 8668815 [details] [diff] [review]:
-----------------------------------------------------------------

having the submitter in its own column appears to waste a lot of horizontal space for most bugs (where the same person submitted all patches), but given one of the goals here is to remove that header row it's probably the best we can do.

::: extensions/MozReview/web/js/mozreview.js
@@ +96,2 @@
>                  for (j = 0; j < family.children.length; j++) {
> +                    tbody.append(rrCommitRow(family.children[j], j==0));

nit: missing spaces around ==

Comment 5

[:glob ✱]

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   6cbec76..4681573  master -> master

and somehow i forgot to fix the == nit before committing this for you:

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   4681573..200cbd8  master -> master