From 3958ebd4985796ca3d1cbc12317c5e003e121bdc Mon Sep 17 00:00:00 2001
From: Patrick Yeo <patrick.yeo@liferay.com>
Date: Tue, 14 Aug 2018 17:13:24 -0700
Subject: [PATCH] Update: (Fixes #1088) List Group variables
 `$list-group-text-link` and `$list-group-subtext-link` should follow same
 pattern as `$list-group-title-link`

---
 .../src/scss/atlas/variables/_list-group.scss | 20 +++++++-------
 .../src/scss/components/_list-group.scss      | 16 ------------
 .../src/scss/variables/_list-group.scss       | 26 ++++++++++---------
 3 files changed, 24 insertions(+), 38 deletions(-)

diff --git a/packages/clay-css/src/scss/atlas/variables/_list-group.scss b/packages/clay-css/src/scss/atlas/variables/_list-group.scss
index 66bae4ae92..12fe64e556 100644
--- a/packages/clay-css/src/scss/atlas/variables/_list-group.scss
+++ b/packages/clay-css/src/scss/atlas/variables/_list-group.scss
@@ -44,28 +44,28 @@ $list-group-subtitle-link: map-merge((
 
 // List Group Text
 
-$list-group-text: () !default;
-$list-group-text: map-merge((
-	line-height: 1.45
-), $list-group-text);
-
 $list-group-text-link: () !default;
 $list-group-text-link: map-merge((
 	hover-color: $body-color
 ), $list-group-text-link);
 
-// List Group Subtext
-
-$list-group-subtext: () !default;
-$list-group-subtext: map-merge((
+$list-group-text: () !default;
+$list-group-text: map-merge((
 	line-height: 1.45
-), $list-group-subtext);
+), $list-group-text);
+
+// List Group Subtext
 
 $list-group-subtext-link: () !default;
 $list-group-subtext-link: map-merge((
 	hover-color: $body-color
 ), $list-group-subtext-link);
 
+$list-group-subtext: () !default;
+$list-group-subtext: map-merge((
+	line-height: 1.45
+), $list-group-subtext);
+
 // List Group Link
 
 $list-group-link-color: $body-color !default;
diff --git a/packages/clay-css/src/scss/components/_list-group.scss b/packages/clay-css/src/scss/components/_list-group.scss
index 06232b3e45..66575b4c2e 100644
--- a/packages/clay-css/src/scss/components/_list-group.scss
+++ b/packages/clay-css/src/scss/components/_list-group.scss
@@ -70,40 +70,24 @@
 	@include clay-text-typography($list-group-title);
 }
 
-// .list-group-title a {
-// 	@include clay-link($list-group-title-link);
-// }
-
 // List Group Subtitle
 
 .list-group-subtitle {
 	@include clay-text-typography($list-group-subtitle);
 }
 
-// .list-group-subtitle a {
-// 	@include clay-link($list-group-subtitle-link);
-// }
-
 // List Group Text
 
 .list-group-text {
 	@include clay-text-typography($list-group-text);
 }
 
-.list-group-text-link {
-	@include clay-link($list-group-text-link);
-}
-
 // List Group Subtext
 
 .list-group-subtext {
 	@include clay-text-typography($list-group-subtext);
 }
 
-.list-group-subtext-link {
-	@include clay-link($list-group-subtext-link);
-}
-
 // List Group Details
 
 .list-group-detail {
diff --git a/packages/clay-css/src/scss/variables/_list-group.scss b/packages/clay-css/src/scss/variables/_list-group.scss
index cbc01078ed..38993f3bbd 100644
--- a/packages/clay-css/src/scss/variables/_list-group.scss
+++ b/packages/clay-css/src/scss/variables/_list-group.scss
@@ -62,34 +62,36 @@ $list-group-subtitle: map-merge((
 
 // List Group Text
 
-$list-group-text: () !default;
-$list-group-text: map-merge((
-	color: $body-color,
-	margin-bottom: 0
-), $list-group-text);
-
 $list-group-text-link: () !default;
 $list-group-text-link: map-merge((
 	color: $body-color,
 	hover-color: darken($body-color, 15%)
 ), $list-group-text-link);
 
+$list-group-text: () !default;
+$list-group-text: map-merge((
+	color: $body-color,
+	margin-bottom: 0,
+	clay-link: $list-group-text-link
+), $list-group-text);
+
 $list-group-text-active-color: $list-group-active-color !default;
 
 // List Group Subtext
 
-$list-group-subtext: () !default;
-$list-group-subtext: map-merge((
-	color: $secondary,
-	margin-bottom: 0
-), $list-group-subtext);
-
 $list-group-subtext-link: () !default;
 $list-group-subtext-link: map-merge((
 	color: $secondary,
 	hover-color: darken($secondary, 15%)
 ), $list-group-subtext-link);
 
+$list-group-subtext: () !default;
+$list-group-subtext: map-merge((
+	color: $secondary,
+	margin-bottom: 0,
+	clay-link: $list-group-subtext-link
+), $list-group-subtext);
+
 $list-group-subtext-active-color: $list-group-active-color !default;
 
 // List Group Detail