opendev/gerrit
Code Issues Proposed changes

Use --dialog-background-color in more dialogs

Browse Source 
Also removes an extra border in the reply dialog.

Change-Id: I5726cae3b4095bdfc7b0647e788923f11d981df4
This commit is contained in:
Kasper Nilsson
2018-04-30 10:28:49 -07:00
parent e46284e4c2
commit bd595d3677
2 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
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;

4
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 {