From ce05f5df5ddaae3f162b8996b0213f6ed9bfc586 Mon Sep 17 00:00:00 2001 From: Johannes Meyer Date: Mon, 17 Mar 2025 17:10:44 +0100 Subject: [PATCH] css: Ensure compatibility with the new table layout --- public/css/common.less | 5 +- public/css/widget/table-layout.less | 72 ----------------------------- 2 files changed, 3 insertions(+), 74 deletions(-) delete mode 100644 public/css/widget/table-layout.less diff --git a/public/css/common.less b/public/css/common.less index f1ed0a08..3a6b5d13 100644 --- a/public/css/common.less +++ b/public/css/common.less @@ -9,7 +9,7 @@ padding-right: 0; .list-item, - .table-layout .table-row { + .table-row { padding-left: 1em; padding-right: 1em; } @@ -257,7 +257,7 @@ div.show-more { } } -.footer { +.content + .footer { display: flex; .box-shadow(0, -1px, 0, 0, @gray-lighter); color: @text-color-light; @@ -316,6 +316,7 @@ div.show-more { } .item-table { + // TODO: Drop once the new item table layout is used everywhere &.table-layout { &.hostgroup-table { --columns: 2; diff --git a/public/css/widget/table-layout.less b/public/css/widget/table-layout.less deleted file mode 100644 index f67ec0f0..00000000 --- a/public/css/widget/table-layout.less +++ /dev/null @@ -1,72 +0,0 @@ -// HostGroup- and -ServiceGroupTable styles - -.item-table.table-layout { - --columns: 1; -} - -ul.item-table.table-layout { - grid-template-columns: 1fr repeat(var(--columns), auto); - - > li { - display: contents; - - &:hover, - &.active { - .col, &::before, &::after { - // The li might get a background on hover. Though, this won't be visible - // as it has no box model since we apply display:contents to it. - background-color: inherit; - } - } - } - - li:not(:last-of-type) { - .col { - border-bottom: 1px solid @gray-light; - } - - .visual { - border-bottom: 1px solid @default-bg; - } - } - - > .table-row { - &:not(:last-of-type) .title .visual { - margin-bottom: ~"calc(-.5em - 1px)"; - } - - .col { - padding: .5em 0; - } - - .col:not(:last-child) { - padding-right: 1em; - } - } -} - -.content.full-width ul.item-table.table-layout { - // Again, since the li has no box model, it cannot have padding. So the first - // and last child need to get the left and right padding respectively. - // But we don't want to have a border that spans to the very right or left, - // so pseudo elements are required. We could add empty cells instead, but - // that would require hard coding the width here, which I'd like to avoid. - - grid-template-columns: ~"auto 1fr repeat(calc(var(--columns) + 1), auto)"; - - > li.table-row { - &::before, &::after { - display: inline-block; - content: '\00a0'; - margin-bottom: 1px; - } - - &::before { - padding-left: inherit; - } - - &::after { - padding-right: inherit; - } - } -}