diff --git a/ui-v2/app/components/consul-service-instance-list/index.hbs b/ui-v2/app/components/consul-service-instance-list/index.hbs index 3756f4b679..df8d294c2f 100644 --- a/ui-v2/app/components/consul-service-instance-list/index.hbs +++ b/ui-v2/app/components/consul-service-instance-list/index.hbs @@ -6,27 +6,37 @@ - + + + {{/if}} \ No newline at end of file diff --git a/ui-v2/app/components/consul-service-list/index.hbs b/ui-v2/app/components/consul-service-list/index.hbs index a877c2c626..f73804825f 100644 --- a/ui-v2/app/components/consul-service-list/index.hbs +++ b/ui-v2/app/components/consul-service-list/index.hbs @@ -4,6 +4,9 @@ {{#let (get proxies item.Name) as |proxy|}} {{#let (service/health-checks item proxy) as |health|}}
+
+ Health +
{{#if (eq 'critical' health)}} @@ -35,40 +38,40 @@ {{/if}} -
    - {{#if (and nspace (env 'CONSUL_NSPACES_ENABLED'))}} - {{#if (not-eq item.Namespace nspace)}} -
  • - {{item.Namespace}} -
  • - {{/if}} - {{/if}} - -
  • - -
  • -
    - -
  • - -
  • -
    - {{#if (not-eq item.InstanceCount 0)}} -
  • - {{format-number item.InstanceCount}} {{pluralize item.InstanceCount 'Instance' without-count=true}} -
  • - {{/if}} - {{#if (get proxies item.Name)}} -
  • +{{#if (and nspace (env 'CONSUL_NSPACES_ENABLED'))}} + {{#if (not-eq item.Namespace nspace)}} +
    +
    + + Namespace + +
    +
    + {{item.Namespace}} +
    +
    + {{/if}} +{{/if}} + + + {{#if (not-eq item.InstanceCount 0)}} + + {{format-number item.InstanceCount}} {{pluralize item.InstanceCount 'Instance' without-count=true}} + + {{/if}} + {{#if (get proxies item.Name)}} +
    +
    + + This service uses a proxy for the Consul service mesh + +
    +
    connected with proxy -
  • - {{/if}} - -
  • - -
  • -
    -
+
+
+ {{/if}} + {{/if}} \ No newline at end of file diff --git a/ui-v2/app/components/tag-list/index.hbs b/ui-v2/app/components/tag-list/index.hbs index ead1355091..da39146b22 100644 --- a/ui-v2/app/components/tag-list/index.hbs +++ b/ui-v2/app/components/tag-list/index.hbs @@ -5,7 +5,11 @@ }} {{else}}
-
Tags
+
+ + Tags + +
{{#each item.Tags as |item|}} {{item}} diff --git a/ui-v2/app/styles/base/components/pill/layout.scss b/ui-v2/app/styles/base/components/pill/layout.scss index f78238ed15..6f97559aec 100644 --- a/ui-v2/app/styles/base/components/pill/layout.scss +++ b/ui-v2/app/styles/base/components/pill/layout.scss @@ -15,9 +15,7 @@ background-color: $gray-100; padding: 0 8px; border-radius: $decor-radius-100; - height: 18px; display: inline-block; - line-height: 0.7rem; } %reduced-pill > span { font-size: 14px; @@ -30,5 +28,4 @@ height: 14px; margin-right: 2px; position: relative; - top: 2px; } diff --git a/ui-v2/app/styles/components/composite-row/layout.scss b/ui-v2/app/styles/components/composite-row/layout.scss index a4bba91ff8..3dd4dfddc9 100644 --- a/ui-v2/app/styles/components/composite-row/layout.scss +++ b/ui-v2/app/styles/components/composite-row/layout.scss @@ -56,21 +56,15 @@ flex-wrap: nowrap; } %composite-row-detail dl, -%composite-row-detail > ul > li { +%composite-row-detail > span { margin-right: 12px; } -%composite-row-detail .node::before, -%composite-row-detail .tag-list::before { - position: relative; -} -%composite-row-detail .node::before { - top: 1px; -} -%composite-row-detail .tag-list::before { - top: 2px; +%composite-row-detail dl.node dt::before { + margin-top: 3px; } %composite-row-detail dl dt::before { margin-right: 4px; + margin-top: 2px; } // Copy Button %composite-row .copy-button button { diff --git a/ui-v2/app/styles/components/composite-row/skin.scss b/ui-v2/app/styles/components/composite-row/skin.scss index 45589a5a74..6bbf81b8b3 100644 --- a/ui-v2/app/styles/components/composite-row/skin.scss +++ b/ui-v2/app/styles/components/composite-row/skin.scss @@ -41,22 +41,22 @@ background-color: var(--brand-600); } // Health Checks -%composite-row-detail li.passing::before, +%composite-row-detail dl.passing dt::before, %composite-row-header .passing dd::before { @extend %with-check-circle-fill-color-mask, %as-pseudo; background-color: $green-500; } -%composite-row-detail li.warning::before, +%composite-row-detail dl.warning dt::before, %composite-row-header .warning dd::before { @extend %with-alert-triangle-color-mask, %as-pseudo; background-color: $orange-500; } -%composite-row-detail li.critical::before, +%composite-row-detail dl.critical dt::before, %composite-row-header .critical dd::before { @extend %with-cancel-square-fill-color-mask, %as-pseudo; background-color: $red-500; } -%composite-row-detail li.empty::before, +%composite-row-detail dl.empty dt::before, %composite-row-header .empty dd::before { @extend %with-minus-square-fill-color-mask, %as-pseudo; background-color: $gray-500; @@ -74,15 +74,15 @@ color: $color-action; text-decoration: underline; } -%composite-row .node::before { +%composite-row dl.node dt::before { @extend %with-git-commit-mask, %as-pseudo; background-color: $gray-500; } -%composite-row .address::before { +%composite-row dl.address dt::before { @extend %with-public-default-mask, %as-pseudo; background-color: $gray-500; } -%composite-row .proxy::before { +%composite-row dl.proxy dt::before { @extend %with-swap-horizontal-mask, %as-pseudo; background-color: $gray-500; } diff --git a/ui-v2/app/styles/components/pill.scss b/ui-v2/app/styles/components/pill.scss index 0d2f09f6d5..71e3b847cf 100644 --- a/ui-v2/app/styles/components/pill.scss +++ b/ui-v2/app/styles/components/pill.scss @@ -13,6 +13,7 @@ span.policy-service-identity::before, span.policy-node-identity::before { display: inline-block; width: auto; + vertical-align: baseline; } span.policy-node-identity::before { content: 'Node Identity: '; diff --git a/ui-v2/app/styles/components/tag-list/layout.scss b/ui-v2/app/styles/components/tag-list/layout.scss index aa037b8fb5..74489afbb2 100644 --- a/ui-v2/app/styles/components/tag-list/layout.scss +++ b/ui-v2/app/styles/components/tag-list/layout.scss @@ -1,14 +1,11 @@ %tag-list { white-space: nowrap; } -%tag-list dt { - display: none; -} // TODO: Currently this is here to overwrite // the default definition list layout used in edit pages // ideally we'd be more specific with those to say // only add padding to dl's in edit pages -%tag-list::before { +%tag-list dt::before { @extend %with-tag-mask, %as-pseudo; margin-right: 4px; }