From 1c8253725d3e776ca854e9540cb85115a72ef926 Mon Sep 17 00:00:00 2001
From: jlukic <jacklukic@gmail.com>
Date: Tue, 16 Sep 2014 21:09:10 -0400
Subject: [PATCH] Adds accordion icon positioning into variables

---
 server/files/javascript/semantic.js                    |  4 ++--
 src/definitions/modules/accordion.less                 |  8 ++++++++
 .../packages/default/modules/accordion.overrides       | 10 ----------
 .../packages/default/modules/accordion.variables       |  8 ++++++++
 4 files changed, 18 insertions(+), 12 deletions(-)
 mode change 100644 => 100755 src/themes/packages/default/modules/accordion.overrides
 mode change 100644 => 100755 src/themes/packages/default/modules/accordion.variables

diff --git a/server/files/javascript/semantic.js b/server/files/javascript/semantic.js
index fbf3c7469..26789b3c5 100755
--- a/server/files/javascript/semantic.js
+++ b/server/files/javascript/semantic.js
@@ -116,12 +116,12 @@ semantic.ready = function() {
               ? $exampleSet.index($lastExample)
               : $exampleSet.size(),
             $examples     = $exampleSet.slice(firstIndex, lastIndex + 1),
-            activeClass   = (index == 0)
+            activeClass   = (index === 0)
               ? 'active '
               : 'active '
           ;
           html += '<div class="item">';
-          html += ' <i class="dropdown icon"></i><a class="'+activeClass+'title"><b>' + $(this).text() + '</b></a>';
+          html += '<a class="'+activeClass+'title"><i class="dropdown icon"></i> <b>' + $(this).text() + '</b></a>';
           html += '<div class="'+activeClass+'content menu">';
           $examples
             .each(function() {
diff --git a/src/definitions/modules/accordion.less b/src/definitions/modules/accordion.less
index c92ce5439..4246eb4c1 100755
--- a/src/definitions/modules/accordion.less
+++ b/src/definitions/modules/accordion.less
@@ -62,9 +62,17 @@
   margin: @iconMargin;
   transition: @iconTransition;
   vertical-align: @iconVerticalAlign;
+  transform: @iconTransform;
 }
 
+/*******************************
+            States
+*******************************/
 
+.ui.accordion .active.title .dropdown.icon,
+.ui.accordion .accordion .active.title .dropdown.icon {
+  transform: @activeIconTransform;
+}
 
 /*******************************
             Types
diff --git a/src/themes/packages/default/modules/accordion.overrides b/src/themes/packages/default/modules/accordion.overrides
old mode 100644
new mode 100755
index 315643b44..c770d78ae
--- a/src/themes/packages/default/modules/accordion.overrides
+++ b/src/themes/packages/default/modules/accordion.overrides
@@ -25,14 +25,4 @@
 .ui.accordion .title .dropdown.icon:before,
 .ui.accordion .accordion .title .dropdown.icon:before {
   content: '\f0da'/*rtl:'\f0d9'*/;
-}
-
-/* Icon Rotation */
-.ui.accordion .title .dropdown.icon,
-.ui.accordion .accordion .title .dropdown.icon {
-  transform: rotate(0deg);
-}
-.ui.accordion .active.title .dropdown.icon,
-.ui.accordion .accordion .active.title .dropdown.icon {
-  transform: rotate(90deg);
 }
\ No newline at end of file
diff --git a/src/themes/packages/default/modules/accordion.variables b/src/themes/packages/default/modules/accordion.variables
old mode 100644
new mode 100755
index c9f1a818e..c52267076
--- a/src/themes/packages/default/modules/accordion.variables
+++ b/src/themes/packages/default/modules/accordion.variables
@@ -24,6 +24,8 @@
 ;
 @iconVerticalAlign: baseline;
 
+@iconTransform: rotate(0deg);
+
 /* Child Accordion */
 @childAccordionMargin: 1em 0em 0em;
 @childAccordionPadding: 0em;
@@ -32,6 +34,12 @@
 @contentMargin: 0em;
 @contentPadding: 0.5em 0em 1em;
 
+/*-------------------
+       States
+--------------------*/
+
+@activeIconTransform: rotate(90deg);
+
 /*-------------------
       Variations
 --------------------*/