The various icons next to filenames are not obvious what they correspond to

NEW
Unassigned

Status

P4
normal
4 years ago
3 years ago

People

(Reporter: jaws, Unassigned)

Tracking

Details

Attachments

(1 attachment)

Created attachment 8550379 [details]
Screenshot of bug

See attached screenshot.

In this screenshot, there are circles with a hole in them. Circles that are filled in green. Circles that are all red with a hole in them. Circles that have some yellow with some green. Circles that have some red with some yellow.

After inspecting the markup I figured out that these corresponded to the types of changes.

All green circle filled in = new file
All green with a hole = preexisting file with only lines added
Yellow means that lines were changed
Red means that lines were removed

With that being said, we should add tooltips to these that say things like "New file", "16 new lines added", "10 lines changed, 3 lines removed", "File removed"

Comment 1

4 years ago
Good points; this should be filed in core.
Flags: needinfo?(mcote)

Comment 2

4 years ago
I've filed this upstream as https://code.google.com/p/reviewboard/issues/detail?id=3812.  I'll keep this bug open until it is fixed (MozReview devs themselves may fix it sometime, but it would be as an upstream patch).

By the way, the Review Board user guide describes the icons at https://www.reviewboard.org/docs/manual/2.0/users/reviews/reviewing-diffs/#the-file-index.
Flags: needinfo?(mcote)

Updated

3 years ago
Priority: -- → P3

Comment 3

3 years ago
We have too many P1s, so I'm spreading out the priorities.  P3 -> P4, P2 -> P3, and some portion of P1s will become P2.
Priority: P3 → P4
(Assignee)

Updated

3 years ago
Product: Developer Services → MozReview
You need to log in before you can comment on or make changes to this bug.