From 6ccb8d595995ed5122cd777850a52ac6a3630ee7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mathieu=20Gagne=CC=81?= Date: Wed, 4 Nov 2015 01:01:38 +0900 Subject: [PATCH] Improve new Gerrit change screen * Add left and right padding around change screen * Add white space between sections * Add table borders to change and commit metadata * Make first line of commit message bold * Make comment's author always bold even when collapsed Change-Id: Ie046f87ea3f875d05a0fd2e3d8be34c906d41d6d --- .../files/gerrit/GerritSite.css | 72 ++++++++++++++++++- 1 file changed, 71 insertions(+), 1 deletion(-) diff --git a/modules/openstack_project/files/gerrit/GerritSite.css b/modules/openstack_project/files/gerrit/GerritSite.css index 5b60381195..5fd22af8e4 100644 --- a/modules/openstack_project/files/gerrit/GerritSite.css +++ b/modules/openstack_project/files/gerrit/GerritSite.css @@ -19,6 +19,76 @@ a.gwt-InlineHyperlink {background: none !important} #gerrit_topmenu .gwt-TextBox {width: 250px} +#change_infoTable { + border-collapse: collapse; +} + +#change_infoTable th { + padding: 2px 4px 2px 6px; + background-color: #eef3f5; + font-style: italic; + text-align: left; +} + +#change_infoTable td { + padding: 2px 4px 2px 6px; + border-bottom: 1px solid #eef3f5; + border-right: 1px solid #eef3f5; +} + +#change_infoTable tr:last-child td { + border: none; +} + +.com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-cs2 { + padding-left: 10px; + padding-right: 10px; +} + +/* Section headers */ +.com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-cs2 + .com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-headerLine, +.com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-cs2 + .com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-sectionHeader { + margin-top: 10px !important; + margin-bottom: 10px !important; +} + +/* Commit message */ +.com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-cs2 + .com-google-gerrit-client-change-CommitBox_BinderImpl_GenCss_style-text::first-line { + font-weight: bold !important; +} + +/* Commit metadata */ +.com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-cs2 + .com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-commitColumn + .com-google-gerrit-client-change-CommitBox_BinderImpl_GenCss_style-header th { + padding: 2px 4px 2px 6px; + background-color: #eef3f5; + font-style: italic; + text-align: left; +} + +.com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-cs2 + .com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-commitColumn + .com-google-gerrit-client-change-CommitBox_BinderImpl_GenCss_style-header td { + border-bottom: 1px solid #eef3f5; + padding: 2px 4px 2px 6px; +} + +.com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-cs2 + .com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-commitColumn + .com-google-gerrit-client-change-CommitBox_BinderImpl_GenCss_style-header td:last-child { + border-right: 1px solid #eef3f5; +} + +/* Review history */ +.com-google-gerrit-client-change-ChangeScreen_BinderImpl_GenCss_style-cs2 + .com-google-gerrit-client-change-Message_BinderImpl_GenCss_style-name { + font-weight: bold !important; +} + .comment_test_name { display: inline-block; *display: inline; @@ -69,4 +139,4 @@ table.infoTable td.notVotable, /* css attribute selector to make -1s show up red in new screen */ [title="This patch needs further work before it can be merged"] { color: red; -} \ No newline at end of file +}