Gerrit revision lists would highly benefit from alternating row colors or highlighting of the hovered row.
Version: unspecified
Severity: normal
• Nikerabbit | |
Jul 2 2012, 8:54 AM |
F9762: Capture_d’écran_2013-02-20_à_14.37.18.png | |
Nov 22 2014, 12:51 AM |
Gerrit revision lists would highly benefit from alternating row colors or highlighting of the hovered row.
Version: unspecified
Severity: normal
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | hashar | T40114 [upstream] alternate change list row colors | |||
Resolved | • demon | T41589 Upgrade Gerrit to 2.6 | |||
Resolved | hashar | T45976 Validate Zuul with Gerrit 2.6 |
I have changed the bug summary which is really about using alternate colors on the changes list.
Timo and Roan did some work to enhance Gerrit look'n feel, seems to be as easy as tweaking a CSS file.
My bet is that this requires change in
gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/ChangeTable2.java
and it's not an easy CSS change (CSS identifiers in GWT change on build)
+upstream (not filed yet)
Actually, I think this will be handled as part of some recent UI changes by OpenStack:
We'll have to set two theme settings (tableOddRowColor & tableEvenRowColor). Blocked on the 2.5 upgraded.
https://gerrit-review.googlesource.com/Documentation/config-gerrit.html#_a_id_theme_a_section_theme
(In reply to comment #9)
white / #DCDCDC ?
Slight different:
#fff / #eee
That way it matches our other white-alternating styles (label columns of info block, header of comment thread etc.)
The color alternance is applied by using a CSS selector :nth-child(odd) and :nth-child(even) on each of the table rows (tr). That is done via the .tableChange class.
On a user dashboard view, the section headers and their changes are all part of the same global table which apply the .tableChange class.
Given a previous section can get an odd or even number of changes, the first change in a section can in turn be either odd or even. Thus we could have the first change of a section to receive the same color as the header :(
We might want to pick a different color to differentiate changes/headers or have the bug fixed upstream. Like pink for headers.
Created attachment 11812
first change line having the same color as the header
Attached:
(In reply to comment #0)
Gerrit patch lists would highly benefit from alternating row colors or
highlighting of the hovered row.
Are you referring to the list of Pathes on a change page
Patch set 3 fd9d49c8 (gitweb)
Patch set 2 4772e9eb (gitweb)
Patch set 1 4ba628db (gitweb)
etc.
Or do you mean "change" lists, the list of changes as a result from a search query or the dashboard for example:
(*) | Change subject | Owner | Project | ...
(*) | Change subject | Owner | Project | ...
(*) | Change subject | Owner | Project | ...
This bug is about alternate change list row colors. So I guess changes and that is what have been implemented upstream. There is no such support for the list of patchset, I guess that should be an other bug.
Christian Aistleitner submitted a patch to upstream: https://gerrit-review.googlesource.com/#/c/42502/
This https://gerrit-review.googlesource.com/#/c/42502/ has been abandoned now.
It was abandoned since it went onto the new change screen in gerrit 2.9.
We run gerrit 2.12 now so we now use the new gerrit change screen.
Is this still a problem or can this be closed and declined or resolved please?
I don't think this is a huge problem, and who knows this may have even been fixed since we last tried. Either way, I restored gerrit 49993
Change 49993 had a related patch set uploaded (by Hashar):
gerrit: alternate change list row colors
@Nikerabbit hi, would you be able to +1 / -1 ^^ please? Just so we know you still want this please.
Mentioned in SAL (#wikimedia-operations) [2017-01-10T21:04:57Z] <mutante> gerrit restarting for config change 49993 (T40114)
Change 331553 had a related patch set uploaded (by Krinkle):
gerrit: Reduce tableEven/OddRowColor contrast.
Looking at the screenshot, I would not have noticed the difference between the rows had I not been looking for it. A middle ground may be appropriate.
Personally, looking at the screenshot, I think the contrast is still larger than it should be. ;)
Can you use different themes? I noticed when viewing a Gerrit diff there is an icon for setting "diff preferences" with a long list of themes from "blackboard" to "cobalt", "lesser_dark" "ruby blue" and lots more. Is that only for diffs or also in other places like change lists? Does it work?
I think that only changes diffs. Please don't mind me, I can deal with it or customize my user CSS :)