From 6258196a97a485db5800c4000a8015c535d942b1 Mon Sep 17 00:00:00 2001
From: jlukic <jacklukic@gmail.com>
Date: Wed, 17 Jun 2015 13:59:24 -0400
Subject: [PATCH] Consolidate and finish work on segment groups

---
 src/definitions/elements/segment.less         | 125 ++++++++++--------
 src/themes/default/elements/segment.variables |  25 +++-
 2 files changed, 96 insertions(+), 54 deletions(-)

diff --git a/src/definitions/elements/segment.less b/src/definitions/elements/segment.less
index 15838130a..e7b1ddc79 100755
--- a/src/definitions/elements/segment.less
+++ b/src/definitions/elements/segment.less
@@ -26,7 +26,7 @@
   position: relative;
   background-color: @background;
   box-shadow: @boxShadow;
-  margin: @margin 0em;
+  margin: @margin;
   padding: @padding;
   border-radius: @borderRadius;
   border: @border;
@@ -56,21 +56,6 @@
   border-bottom: none;
 }
 
-/* Horizontal */
-.ui.horizontal.segment,
-.ui.horizontal.segments > .segment {
-  margin: 0em;
-  padding-top: 0em;
-  padding-bottom: 0em;
-  min-width: 0px;
-
-  background-color: transparent;
-  border-radius: 0px;
-  border: none;
-  box-shadow: none;
-  border-right: @borderWidth solid @borderColor;
-}
-
 
 /*-------------------
     Loose Coupling
@@ -106,7 +91,7 @@
   padding-bottom: @pageGridMargin;
 }
 .ui.grid.segment {
-  margin: @margin 0rem;
+  margin: @margin;
   border-radius: @borderRadius;
 }
 
@@ -275,10 +260,13 @@
             Groups
 *******************************/
 
+/* Group */
 .ui.segments {
   position: relative;
-  margin: @margin 0em;
-  box-shadow: @boxShadow;
+  margin: @groupedMargin;
+  border: @groupedBorder;
+  box-shadow: @groupedBoxShadow;
+  border-radius: @groupedBorderRadius;
 }
 .ui.segments:first-child {
   margin-top: 0em;
@@ -287,59 +275,90 @@
   margin-bottom: 0em;
 }
 
+
+/* Nested Segment */
 .ui.segments > .segment {
   top: 0px;
   bottom: 0px;
-  margin: 0em @attachedHorizontalOffset;
-  width: @attachedWidth;
-  max-width: @attachedWidth;
   border-radius: 0px;
-  box-shadow: @attachedBoxShadow;
-  border: @attachedBorder;
+  margin: @groupedSegmentMargin;
+  width: @groupedSegmentWidth;
+  box-shadow: @groupedSegmentBoxShadow;
+  border: @groupedSegmentBorder;
+  border-top: @groupedSegmentDivider;
 }
 
-.ui.segments > .segment:not(:last-child) {
-  border-bottom: none;
+.ui.segments:not(.horizontal) > .segment:first-child {
+  top: @attachedTopOffset;
+  bottom: 0px;
+  border-top: none;
+  margin-top: 0em;
+  bottom: 0px;
+  margin-bottom: 0em;
+  top: @attachedTopOffset;
+  border-radius: @borderRadius @borderRadius 0em 0em;
 }
 
+/* Bottom */
+.ui.segments:not(.horizontal) > .segment:last-child {
+  top: @attachedBottomOffset;
+  bottom: 0px;
+  margin-top: 0em;
+  margin-bottom: 0em;
+  box-shadow: @attachedBottomBoxShadow;
+  border-radius: 0em 0em @borderRadius @borderRadius;
+}
+
+
+/* Nested Group */
+.ui.segments > .ui.segments {
+  border-top: @groupedSegmentDivider;
+  margin: @nestedGroupMargin;
+}
+.ui.segments > .segments:first-child {
+  border-top: none;
+}
+.ui.segments > .segment + .segments:not(.horizontal) {
+  margin-top: 0em;
+}
 
 /* Horizontal Group */
 .ui.horizontal.segments {
   display: flex;
   flex-direction: row;
-  padding: @padding;
+  background-color: transparent;
+  border-radius: 0px;
+  padding: 0em;
   background-color: @background;
   box-shadow: @boxShadow;
-  margin: @margin 0em;
-  padding: @padding;
+  margin: @margin;
   border-radius: @borderRadius;
   border: @border;
 }
-.ui.horizontal.segments > .segment:first-child {
-  padding-left: 0px;
-}
-.ui.horizontal.segments > .segment:last-child {
-  padding-right: 0px;
-  border-right: none;
-}
 
-/* Top */
-.ui.segments > .segment:first-child {
-  margin-top: 0em;
-  bottom: 0px;
-  margin-bottom: 0em;
-  top: @attachedTopOffset;
-  border-radius: @borderRadius @borderRadius 0em 0em;
+/* Nested Horizontal Group */
+.ui.segments > .horizontal.segments {
+  margin: 0em;
+  background-color: transparent;
+  border-radius: 0px;
+  border: none;
+  box-shadow: none;
+  border-top: @groupedSegmentDivider;
 }
 
-/* Bottom */
-.ui.segments > .segment:last-child {
-  bottom: 0px;
-  margin-top: 0em;
-  margin-bottom: 0em;
-  top: @attachedBottomOffset;
-  box-shadow: @attachedBottomBoxShadow;
-  border-radius: 0em 0em @borderRadius @borderRadius;
+/* Horizontal Segment */
+.ui.horizontal.segments > .segment {
+  flex: 1 1 auto;
+  margin: 0em;
+  min-width: 0px;
+  background-color: transparent;
+  border-radius: 0px;
+  border: none;
+  box-shadow: none;
+  border-left: @borderWidth solid @borderColor;
+}
+.ui.horizontal.segments > .segment:first-child {
+  border-left: none;
 }
 
 
@@ -583,11 +602,11 @@
 .ui.floated.segment,
 .ui[class*="left floated"].segment {
   float: left;
-  margin-right: @margin;
+  margin-right: @floatedDistance;
 }
 .ui[class*="right floated"].segment {
   float: right;
-  margin-left: @margin;
+  margin-left: @floatedDistance;
 }
 
 
diff --git a/src/themes/default/elements/segment.variables b/src/themes/default/elements/segment.variables
index 44e10df6e..bf8f9a9b0 100644
--- a/src/themes/default/elements/segment.variables
+++ b/src/themes/default/elements/segment.variables
@@ -15,9 +15,29 @@
 @horizontalPadding: 1em;
 @padding: @verticalPadding @horizontalPadding;
 
-@margin: 1rem;
+@verticalMargin: 1rem;
+@horizontalMargin: 0em;
+@margin: @verticalMargin @horizontalMargin;
 @borderRadius: @defaultBorderRadius;
 
+/*-------------------
+       Group
+--------------------*/
+
+@groupedMargin: @margin;
+@groupedBorder: @border;
+@groupedBoxShadow: @boxShadow;
+@groupedBorderRadius: @borderRadius;
+
+@nestedGroupMargin: @verticalMargin @verticalMargin;
+
+@groupedSegmentBorder: none;
+@groupedSegmentDivider: @border;
+@groupedSegmentMargin: 0em;
+@groupedSegmentWidth: auto;
+@groupedSegmentBoxShadow: none;
+
+
 /*-------------------
        Coupling
 --------------------*/
@@ -81,6 +101,9 @@
   @subtleShadow
 ;
 
+/* Floated */
+@floatedDistance: 1em;
+
 /* Basic */
 @basicBackground: none transparent;
 @basicBorder: none;