opendev/gerrit
Code Issues Proposed changes

Merge changes I5726cae3,Ie3acbfa4

Browse Source 
* changes:
  Use --dialog-background-color in more dialogs
  Rework some CSS constant usage
This commit is contained in:
Wyatt Allen
2018-05-01 01:40:00 +00:00
committed by Gerrit Code Review
parent 26953b58e2 bd595d3677
commit 12f92ebdae
6 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
polygerrit-ui/app/elements/change/gr-included-in-dialog/gr-included-in-dialog.html
View File

@@ -23,13 +23,13 @@ limitations under the License.
<template> <template>
<style include="shared-styles"> <style include="shared-styles">
:host { :host {
background-color: var(--dialog-background-color);
display: block; display: block;
max-height: 80vh; max-height: 80vh;
overflow-y: auto; overflow-y: auto;
padding: 4.5em 1em 1em 1em; padding: 4.5em 1em 1em 1em;
} }
header { header {
background: var(--view-background-color);
border-bottom: 1px solid var(--border-color); border-bottom: 1px solid var(--border-color);
left: 0; left: 0;
padding: 1em; padding: 1em;
@@ -58,8 +58,9 @@ limitations under the License.
margin-bottom: 1em; margin-bottom: 1em;
} }
ul li { ul li {
border: 1px solid var(--border-color);
border-radius: .2em; border-radius: .2em;
background: var(--header-background-color); background: var(--chip-background-color);
display: inline-block; display: inline-block;
margin: 0 .2em .4em .2em; margin: 0 .2em .4em .2em;
padding: .2em .4em; padding: .2em .4em;

6
polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.html
View File

@@ -38,6 +38,7 @@ limitations under the License.
<template> <template>
<style include="shared-styles"> <style include="shared-styles">
:host { :host {
background-color: var(--dialog-background-color);
display: block; display: block;
max-height: 100%; max-height: 100%;
} }
@@ -59,7 +60,7 @@ limitations under the License.
width: 100%; width: 100%;
} }
.actions { .actions {
background-color: var(--view-background-color); background-color: var(--dialog-background-color);
bottom: 0; bottom: 0;
display: flex; display: flex;
justify-content: space-between; justify-content: space-between;
@@ -75,6 +76,7 @@ limitations under the License.
flex-shrink: 0; flex-shrink: 0;
} }
.peopleContainer { .peopleContainer {
border-top: none;
display: table; display: table;
} }
.peopleList { .peopleList {
@@ -120,7 +122,7 @@ limitations under the License.
display: block; display: block;
} }
.previewContainer gr-formatted-text { .previewContainer gr-formatted-text {
background: var(--header-background-color); background: var(--table-header-background-color);
padding: 1em; padding: 1em;
} }
.draftsContainer h3 { .draftsContainer h3 {

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

@@ -202,7 +202,7 @@ limitations under the License.
display: block; display: block;
} }
.target-row td.blame { .target-row td.blame {
background: var(--header-background-color); background: var(--diff-selection-background-color);
} }
col.blame { col.blame {
display: none; display: none;

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

@@ -31,7 +31,7 @@ limitations under the License.
} }
.container { .container {
align-items: center; align-items: center;
background: var(--header-background-color); background: var(--table-header-background-color);
border-radius: .75em; border-radius: .75em;
display: inline-flex; display: inline-flex;
padding: 0 .5em; padding: 0 .5em;

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

@@ -31,7 +31,7 @@ limitations under the License.
} }
.container { .container {
align-items: center; align-items: center;
background: var(--header-background-color); background: var(--chip-background-color);
border-radius: .75em; border-radius: .75em;
display: inline-flex; display: inline-flex;
padding: 0 .5em; padding: 0 .5em;

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

@@ -42,6 +42,8 @@ limitations under the License.
--table-header-background-color: #fafafa; --table-header-background-color: #fafafa;
--table-subheader-background-color: #eaeaea; --table-subheader-background-color: #eaeaea;
--chip-background-color: var(--header-background-color);
--dropdown-background-color: #fff; --dropdown-background-color: #fff;
/* Font sizes */ /* Font sizes */