opendev/gerrit
Code Issues Proposed changes

Define table-subheader-background-color

Browse Source 
Defines a CSS constant and uses it throughout the app.

Change-Id: I610e8eb4a51aa72e8eea526c585f308cabb55169
This commit is contained in:
Kasper Nilsson
2018-04-16 11:52:50 +02:00
parent 56628163af
commit 8add6730c8
4 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
polygerrit-ui/app/elements/shared/gr-button/gr-button.html
View File

@@ -72,7 +72,7 @@ limitations under the License.
/* Keep below color definition for primary/secondary so that this takes /* Keep below color definition for primary/secondary so that this takes
precedence when disabled. */ precedence when disabled. */
:host([disabled]) { :host([disabled]) {
--background-color: #eaeaea; --background-color: var(--table-subheader-background-color);
--button-color: #a8a8a8; --button-color: #a8a8a8;
cursor: default; cursor: default;
} }

2
polygerrit-ui/app/elements/shared/gr-page-nav/gr-page-nav.html
View File

@@ -22,7 +22,7 @@ limitations under the License.
<template> <template>
<style include="shared-styles"> <style include="shared-styles">
#nav { #nav {
background-color: #f5f5f5; background-color: var(--table-header-background-color);
border: 1px solid var(--border-color); border: 1px solid var(--border-color);
border-top: none; border-top: none;
height: 100%; height: 100%;

1
polygerrit-ui/app/styles/app-theme.html
View File

@@ -36,6 +36,7 @@ limitations under the License.
transition: none; transition: none;
} }
--table-header-background-color: #fafafa; --table-header-background-color: #fafafa;
--table-subheader-background-color: #eaeaea;
/* Font sizes */ /* Font sizes */
--font-size-normal: 1rem; --font-size-normal: 1rem;

2
polygerrit-ui/app/styles/gr-change-list-styles.html
View File

@@ -80,7 +80,7 @@ limitations under the License.
top: 0; top: 0;
} }
.groupHeader { .groupHeader {
background-color: #eaeaea; background-color: var(--table-subheader-background-color);
} }
.groupHeader a { .groupHeader a {
color: var(--primary-text-color); color: var(--primary-text-color);