opendev/gerrit
Code Issues Proposed changes

Some further layout/design tweaks

Browse Source 
Just some fixes and tweaks discovered when thoroughly navigating through
*all* the pages (settings, admin, ...).

Change-Id: I157db71c73671b6e765b1714ce05dbd376082971
This commit is contained in:
Ben Rohlfs
2020-02-19 15:11:43 +01:00
parent e6fb017fa0
commit 0b4e4e7658
7 changed files with 16 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
polygerrit-ui/app/styles/gr-menu-page-styles.html
View File

@@ -18,7 +18,6 @@ limitations under the License.
<template>
<style>
:host {
background-color: var(--view-background-color);
display: block;
}
main {

21
polygerrit-ui/app/styles/gr-table-styles.html
View File

@@ -19,12 +19,13 @@ limitations under the License.
<template>
<style>
.genericList {
background-color: var(--background-color-primary);
border-collapse: collapse;
width: 100%;
}
.genericList th,
.genericList td {
height: 2.25rem;
padding: var(--spacing-s) 0;
padding: var(--spacing-m) 0;
vertical-align: middle;
}
.genericList tr {
@@ -51,11 +52,17 @@ limitations under the License.
.genericList tr td:last-of-type {
border-left: 1px solid var(--border-color);
text-align: center;
padding: 0 var(--spacing-l);
padding-left: var(--spacing-l);
}
.genericList tr th.delete,
.genericList tr td.delete {
padding-top: 0;
padding-bottom: 0;
}
.genericList tr th.delete,
.genericList tr td.delete,
.genericList tr.loadingMsg td {
.genericList tr.loadingMsg td,
.genericList tr.groupHeader td {
border-left: none;
}
.genericList .loading {
@@ -72,12 +79,8 @@ limitations under the License.
text-align: left;
vertical-align: middle
}
.genericList .topHeader {
background-color: var(--table-header-background-color);
height: 3rem;
}
.genericList .groupHeader {
background-color: var(--table-subheader-background-color);
background-color: var(--background-color-secondary);
font-family: var(--header-font-family);
font-size: var(--font-size-h3);
font-weight: var(--font-weight-h3);

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

@@ -40,7 +40,7 @@ limitations under the License.
box-sizing: border-box;
}
input {
background-color: inherit;
background-color: var(--background-color-primary);
border: 1px solid var(--border-color);
border-radius: var(--border-radius);
box-sizing: border-box;

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

@@ -62,6 +62,7 @@ html {
--edit-mode-background-color: #ebf5fb;
--emphasis-color: #fff9c4;
--hover-background-color: rgba(161, 194, 250, 0.2);
--disabled-button-background-color: #e8eaed;
--primary-button-background-color: #2a66d9;
--selection-background-color: rgba(161, 194, 250, 0.1);
--tooltip-background-color: #333;

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

@@ -52,6 +52,7 @@ limitations under the License.
--edit-mode-background-color: #5c0a36;
--emphasis-color: #383f4a;
--hover-background-color: rgba(161, 194, 250, 0.2);
--disabled-button-background-color: #484a4d;
--primary-button-background-color: var(--link-color);
--selection-background-color: rgba(161, 194, 250, 0.1);
--tooltip-background-color: #111;