Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use variables to fix style and dark theme issue, fix #380 #381

Merged
merged 2 commits into from
Sep 2, 2020

Conversation

jancborchardt
Copy link
Member

@axheli this fixes the issues #380 you reported, please review. :)

@AndyXheli
Copy link

@jancborchardt

same issue

image

@jancborchardt
Copy link
Member Author

@axheli did you check out this branch for testing, or did you manually apply the changes? It should absolutely work, as there’s no set white color anywhere anymore.

@AndyXheli
Copy link

AndyXheli commented Aug 25, 2020

@jancborchardt Yes i did i applied the setting and rebooted the server and cleared the web cache.

@AndyXheli
Copy link

.content {
width: 100%;
}

#logreader-root {
height: calc(100vh - 50px);
width: 100%;
}

#logreader-root > div {
height: 100%;
}

#log-section {
display: none; /** we overwrite this */
}

.log-loading {
position: absolute;
top: 50%;
left: 50%;
transform: translate(-50%, -50%);
}

.scrollContainer {
height: 100%;
overflow: auto;
}

.highlight-enter {
background-color: var(--color-warning);
}

.highlight-enter.highlight-enter-active {
background-color: var(--color-main-background);
transition: background 1500ms ease-in;
}

@AndyXheli
Copy link

.logs {
width: 100%;
white-space: normal;
margin-bottom: 14px;
table-layout: fixed;

th {
color: #888;
}

td, th {
vertical-align: top;
padding: .8em;
text-align: left;
font-weight: normal;
position: relative;
word-wrap: break-word;
border-bottom: 1px solid #eee;
}

th.level.active .log-settings-toggle,
th.level:hover .log-settings-toggle {
opacity: 1;
}

.time {
width: 180px;
}

.time.relative {
width: 100px;
}

.app {
width: 100px;
white-space: nowrap;
text-overflow: ellipsis;
overflow: hidden;
}

td.app, th.app {
@media only screen and (max-width: 1100px) {
display: none;
}
}

.level {
width: 100px;
}

/* warning*/
.level_2 td.level {
border-left: 5px solid #FEEFB3;
}

/error/
.level_3 td.level {
border-left: 5px solid #FFBABA;
}

/fatal/
.level_4 td.level {
border-left: 5px solid #ff8080;
}

div.column {
box-sizing: border-box;
padding: 7px;
}

div.level {
font-weight: bold;
}

div.level, div.app, div.time {
display: inline-block;
width: 33%;
}

div.time {
padding-right: 20px;
}

div.row {
position: relative;
}

div.message {
border-bottom: 1px solid #888;
}

td.copy, th.copy {
position: relative;
width: 42px;
margin: 0;
overflow: visible;

button {
  position: relative;
  opacity: 0;
  transition: opacity 0.5s;
  top: -5px;
  padding: 0;
  background-color: transparent;
  border: none;
  width: 42px;
}

}

tr:hover {
td.copy button {
opacity: 0.5;

  &:hover {
    opacity: 1;
  }
}

}

tr.active td.copy button {
opacity: 1;
}

div.level_1 {

}

div.level_2 {
border-left: 5px solid #FEEFB3;
}

div.level_3 {
border-left: 5px solid #FFBABA;
}

div.level_4 {
border-left: 5px solid #ff8080;
}

tr.empty td {
border-bottom: none;
}
tr.highlight {
background-color: var(--color-background-hover);
}
}

.smallHeader,
th.level {
overflow: visible;

span {
margin: -0.8em;
padding: .8em;
display: block;
width: 100%;
height: 100%;
cursor: pointer;
}

.log-settings-toggle {
cursor: pointer;
display: block;
float: right;
margin: 2px;
opacity: 0.5;
transition: opacity 0.2s;
}
}

.smallHeader {
margin-left: 44px;
width: 62px;
padding: 10px;

.log-settings-toggle:hover {
opacity: 1;
}
}

.copyMenu {
right: -10px;
top: 50px;
width: 150px;
}

Signed-off-by: Jan C. Borchardt <hey@jancborchardt.net>
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 merged commit 262ca47 into master Sep 2, 2020
@delete-merged-branch delete-merged-branch bot deleted the design/variables branch September 2, 2020 14:52
@rullzer rullzer mentioned this pull request Sep 3, 2020
21 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants