From bf1932dccf8d95d8be2b6f750da8e696ee7849c1 Mon Sep 17 00:00:00 2001 From: leigh-mil Date: Thu, 15 Aug 2019 13:46:41 -0400 Subject: [PATCH 1/4] Add padding around the text in remove button --- styles/components/_accordion_table.scss | 4 ---- 1 file changed, 4 deletions(-) diff --git a/styles/components/_accordion_table.scss b/styles/components/_accordion_table.scss index d9e740af..f4550262 100644 --- a/styles/components/_accordion_table.scss +++ b/styles/components/_accordion_table.scss @@ -119,10 +119,6 @@ float: none; } - > *:first-child { - padding-left: 0; - } - > *:last-child { float: right; } From 16c764627ec04f946cbb569e22c712f124e91592 Mon Sep 17 00:00:00 2001 From: leigh-mil Date: Thu, 15 Aug 2019 13:54:06 -0400 Subject: [PATCH 2/4] Use text align instead of float to position remove button --- styles/components/_accordion_table.scss | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/styles/components/_accordion_table.scss b/styles/components/_accordion_table.scss index f4550262..5c3f3802 100644 --- a/styles/components/_accordion_table.scss +++ b/styles/components/_accordion_table.scss @@ -107,6 +107,7 @@ &__action-group { padding: 1rem 3.2rem; background-color: $color-gray-lightest; + text-align: right; button, a { @@ -118,10 +119,6 @@ padding-top: 0.5rem; float: none; } - - > *:last-child { - float: right; - } } &__toggler { From 3026b41c4ef1dd618537e12fd7cba15ccec3120d Mon Sep 17 00:00:00 2001 From: leigh-mil Date: Thu, 15 Aug 2019 14:16:19 -0400 Subject: [PATCH 3/4] Move app name and description to their own line --- styles/elements/_inputs.scss | 4 ++++ templates/portfolios/applications/settings.html | 8 ++++---- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/styles/elements/_inputs.scss b/styles/elements/_inputs.scss index 660bef04..2845fe43 100644 --- a/styles/elements/_inputs.scss +++ b/styles/elements/_inputs.scss @@ -98,6 +98,10 @@ cursor: default; margin: 0 0 0 ($gap / 2); } + + &__view-only { + margin: 0; + } } &__help { diff --git a/templates/portfolios/applications/settings.html b/templates/portfolios/applications/settings.html index b9bea61f..42c494ad 100644 --- a/templates/portfolios/applications/settings.html +++ b/templates/portfolios/applications/settings.html @@ -56,18 +56,18 @@

-
+
{{ application_form.name.label() }} - {{ application_form.name.data }}
+ {{ application_form.name.data }}
-
+
{{ application_form.description.label() }} - {{ application_form.description.data }}
+ {{ application_form.description.data }}
From 4660bc50eba86d4bcece42015b459787c20122cb Mon Sep 17 00:00:00 2001 From: leigh-mil Date: Thu, 15 Aug 2019 14:21:10 -0400 Subject: [PATCH 4/4] Remove bullet points from li elements in accordion tables --- styles/components/_accordion_table.scss | 1 + 1 file changed, 1 insertion(+) diff --git a/styles/components/_accordion_table.scss b/styles/components/_accordion_table.scss index 5c3f3802..ba0718d1 100644 --- a/styles/components/_accordion_table.scss +++ b/styles/components/_accordion_table.scss @@ -49,6 +49,7 @@ font-weight: $font-bold; border-bottom: 1px solid $color-gray-lightest; margin: 0; + list-style-type: none; .icon-link--large { padding-top: $gap * 0.5;