From 611e3d915721f704785028ad60297f2dfb6f2f4e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mathieu=20Gagne=CC=81?= Date: Tue, 5 Jan 2016 13:55:01 -0500 Subject: [PATCH] gerrit: Add borders to test result table The test result table on old change screen used to have borders. This change adds them back so one can easily read test result with long job name. It removes need to make extra mental effort to align job name to their result when table is overly long or large. It also supports use case where Zuul inline status is enabled. Borders won't span across both Zuul inline status and test result table. Change-Id: I199a7698304772b48683966d8d7d3a16c7a6924f --- .../files/gerrit/GerritSite.css | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/modules/openstack_project/files/gerrit/GerritSite.css b/modules/openstack_project/files/gerrit/GerritSite.css index 5dd7fd83c5..edb998f6b1 100644 --- a/modules/openstack_project/files/gerrit/GerritSite.css +++ b/modules/openstack_project/files/gerrit/GerritSite.css @@ -141,6 +141,24 @@ table.infoTable td.notVotable, background: inherit; } +.test_result_table { + border-collapse: collapse; +} + +.test_result_table tr { + border-left: 1px solid #eef3f5; + border-right: 1px solid #eef3f5; +} + +.test_result_table td.header { + background-color: #eef3f5; +} + +.test_result_table td { + padding: 2px 4px 2px 6px; + border: 1px solid #eef3f5; +} + .addMemberTextBox { width: 20em; }