opendev/gerrit
Code Issues Proposed changes

Update file status colors

Browse Source 
Colors will be more consistent with vote chips, merged status colors.

Change-Id: I3ce4943a5dae95bdf361c8f6981cbb27c9067b74
This commit is contained in:
Milutin Kristofic
2021-03-30 23:35:30 +02:00
parent 342eb149a9
commit 0205d1b231
3 changed files with 13 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
polygerrit-ui/app/elements/shared/gr-file-status-chip/gr-file-status-chip_html.ts
View File

@@ -25,7 +25,7 @@ export const htmlTemplate = html`
padding: 0 var(--spacing-m);
color: var(--primary-text-color);
font-size: var(--font-size-small);
background-color: var(--dark-add-highlight-color);
background-color: var(--file-status-added);
}
.status.invisible,
.status.M {
@@ -34,10 +34,10 @@ export const htmlTemplate = html`
.status.D,
.status.R,
.status.W {
background-color: var(--dark-remove-highlight-color);
background-color: var(--file-status-changed);
}
.status.U {
background-color: var(--comment-background-color);
background-color: var(--file-status-unchanged);
}
</style>
<span

5
polygerrit-ui/app/styles/themes/app-theme.ts
View File

@@ -257,6 +257,11 @@ const styleSheet = safeStyleSheet`
--status-ready: var(--pink-800);
--status-custom: var(--purple-900);
/* file status colors */
--file-status-added: var(--green-300);
--file-status-changed: var(--red-200);
--file-status-unchanged: var(--grey-300);
/* fonts */
--font-family: 'Roboto', -apple-system, BlinkMacSystemFont, 'Segoe UI', Helvetica, Arial, sans-serif, 'Apple Color Emoji', 'Segoe UI Emoji', 'Segoe UI Symbol';
--header-font-family: 'Open Sans', 'Roboto', -apple-system, BlinkMacSystemFont, 'Segoe UI', Helvetica, Arial, sans-serif, 'Apple Color Emoji', 'Segoe UI Emoji', 'Segoe UI Symbol';

5
polygerrit-ui/app/styles/themes/dark-theme.ts
View File

@@ -155,6 +155,11 @@ function getStyleEl() {
--status-ready: var(--pink-500);
--status-custom: var(--purple-400);
/* file status colors */
--file-status-added: var(--green-tonal);
--file-status-changed: var(--red-tonal);
--file-status-unchanged: var(--grey-700);
/* fonts */
--font-weight-bold: 700; /* 700 is the same as 'bold' */