From: Geoff Kimball <geoff@zurb.com>
Date: Tue, 24 Nov 2015 19:03:06 +0000 (-0800)
Subject: Fix pagination arrow settings not working
X-Git-Tag: v6.0.4~14
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=08e6c344933e24ca974d9458b069c94d66aee336;p=thirdparty%2Ffoundation%2Ffoundation-sites.git

Fix pagination arrow settings not working
---

diff --git a/docs/pages/pagination.md b/docs/pages/pagination.md
index 1f2e8853f..8d7f62e35 100644
--- a/docs/pages/pagination.md
+++ b/docs/pages/pagination.md
@@ -14,7 +14,7 @@ Extra screen reader-only text should also be added to a pagination element. In t
 
 ```html_example
 <ul class="pagination" role="navigation" aria-label="Pagination">
-  <li class="disabled">Previous <span class="show-for-sr">page</span></li>
+  <li class="pagination-previous disabled">Previous <span class="show-for-sr">page</span></li>
   <li class="current"><span class="show-for-sr">You're on page</span> 1</li>
   <li><a href="#" aria-label="Page 2">2</a></li>
   <li><a href="#" aria-label="Page 3">3</a></li>
@@ -22,7 +22,7 @@ Extra screen reader-only text should also be added to a pagination element. In t
   <li class="ellipsis" aria-hidden="true"></li>
   <li><a href="#" aria-label="Page 12">12</a></li>
   <li><a href="#" aria-label="Page 13">13</a></li>
-  <li><a href="#" aria-label="Next page">Next <span class="show-for-sr">page</span></a></li>
+  <li class="pagination-next"><a href="#" aria-label="Next page">Next <span class="show-for-sr">page</span></a></li>
 </ul>
 ```
 
@@ -34,7 +34,7 @@ The items in a pagination list are `display: inline-block`, which makes centerin
 
 ```html_example
 <ul class="pagination text-center" role="navigation" aria-label="Pagination">
-  <li class="disabled">Previous</li>
+  <li class="pagination-previous disabled">Previous</li>
   <li class="current"><span class="show-for-sr">You're on page</span> 1</li>
   <li><a href="#" aria-label="Page 2">2</a></li>
   <li><a href="#" aria-label="Page 3">3</a></li>
@@ -42,6 +42,6 @@ The items in a pagination list are `display: inline-block`, which makes centerin
   <li class="ellipsis"></li>
   <li><a href="#" aria-label="Page 12">12</a></li>
   <li><a href="#" aria-label="Page 13">13</a></li>
-  <li><a href="#" aria-label="Next page">Next</a></li>
+  <li class="pagination-next"><a href="#" aria-label="Next page">Next</a></li>
 </ul>
 ```
diff --git a/scss/components/_pagination.scss b/scss/components/_pagination.scss
index 72aba4e57..4d4557779 100644
--- a/scss/components/_pagination.scss
+++ b/scss/components/_pagination.scss
@@ -95,25 +95,6 @@ $pagination-arrows: true !default;
       background: $pagination-item-background-hover;
     }
   }
-
-  // Symbols
-  @if $pagination-arrows {
-    [aria-label='previous'] {
-      &::before {
-        content: '«';
-        display: inline-block;
-        margin-#{$global-right}: 0.75rem;
-      }
-    }
-
-    [aria-label='next'] {
-      &::after {
-        content: '»';
-        display: inline-block;
-        margin-#{$global-left}: 0.75rem;
-      }
-    }
-  }
 }
 
 /// Adds styles for the current pagination item. Apply this to an `<a>`.
@@ -158,4 +139,20 @@ $pagination-arrows: true !default;
       @include pagination-ellipsis;
     }
   }
+
+  @if $pagination-arrows {
+    .pagination-previous a::before,
+    .pagination-previous.disabled::before {
+      content: '«';
+      display: inline-block;
+      margin-#{$global-right}: 0.5rem;
+    }
+
+    .pagination-next a::after,
+    .pagination-next.disabled::after {
+      content: '»';
+      display: inline-block;
+      margin-#{$global-left}: 0.5rem;
+    }
+  }
 }