From 344a3bb99f342f2a60cbb61d6dc1cc32b5d70b0f Mon Sep 17 00:00:00 2001
From: Stacey Gammon <gammon@elastic.co>
Date: Tue, 16 May 2017 14:33:56 -0400
Subject: [PATCH] Address code review comments

---
 ui_framework/components/pager/pager.js                | 11 ++++++-----
 ui_framework/components/pager/pager_button_group.js   |  2 ++
 ui_framework/doc_site/src/services/routes/routes.js   |  1 +
 .../doc_site/src/views/pager/pager_example.js         |  4 ++--
 4 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/ui_framework/components/pager/pager.js b/ui_framework/components/pager/pager.js
index 78fb61d088c4b..c485091a00d4c 100644
--- a/ui_framework/components/pager/pager.js
+++ b/ui_framework/components/pager/pager.js
@@ -12,13 +12,14 @@ export function KuiPager({
     hasNextPage,
     onNextPage,
     onPreviousPage,
-    ...rest }) {
+    ...rest,
+  }) {
   const classes = classNames('kuiPager', className);
   return (
     <div className={classes} { ...rest }>
       <div className="kuiPagerText">{startNumber}&ndash;{endNumber} of {totalItems}</div>
       {
-        startNumber === 1 && endNumber === totalItems
+        (startNumber === 1 && endNumber === totalItems)
           ? null
           : <KuiPagerButtonGroup
               hasNext={hasNextPage}
@@ -32,9 +33,9 @@ export function KuiPager({
 }
 
 KuiPager.propTypes = {
-  startNumber: React.PropTypes.number,
-  endNumber: React.PropTypes.number,
-  totalItems: React.PropTypes.number,
+  startNumber: React.PropTypes.number.isRequired,
+  endNumber: React.PropTypes.number.isRequired,
+  totalItems: React.PropTypes.number.isRequired,
   hasPreviousPage: React.PropTypes.bool.isRequired,
   hasNextPage: React.PropTypes.bool.isRequired,
   onNextPage: React.PropTypes.func.isRequired,
diff --git a/ui_framework/components/pager/pager_button_group.js b/ui_framework/components/pager/pager_button_group.js
index 7d40d47625127..e6a34b6b4c8ff 100644
--- a/ui_framework/components/pager/pager_button_group.js
+++ b/ui_framework/components/pager/pager_button_group.js
@@ -6,6 +6,7 @@ export function KuiPagerButtonGroup({ className, onPrevious, onNext, hasNext, ha
   return (
     <KuiButtonGroup isUnited className={className} { ...rest }>
       <KuiButton
+        aria-label="Show previous page"
         data-test-subj="pagerPreviousButton"
         type="basic"
         onClick={onPrevious}
@@ -13,6 +14,7 @@ export function KuiPagerButtonGroup({ className, onPrevious, onNext, hasNext, ha
         icon={<KuiButtonIcon type="previous" />}
       />
       <KuiButton
+        aria-label="Show next page"
         data-test-subj="pagerNextButton"
         type="basic"
         onClick={onNext}
diff --git a/ui_framework/doc_site/src/services/routes/routes.js b/ui_framework/doc_site/src/services/routes/routes.js
index d4f480bdcc121..442ebc1a786b9 100644
--- a/ui_framework/doc_site/src/services/routes/routes.js
+++ b/ui_framework/doc_site/src/services/routes/routes.js
@@ -173,6 +173,7 @@ const components = [{
 }, {
   name: 'Pager',
   component: PagerExample,
+  hasReact: true,
 }, {
   name: 'Panel',
   component: PanelExample,
diff --git a/ui_framework/doc_site/src/views/pager/pager_example.js b/ui_framework/doc_site/src/views/pager/pager_example.js
index bbda41824d96a..9e9101e383b25 100644
--- a/ui_framework/doc_site/src/views/pager/pager_example.js
+++ b/ui_framework/doc_site/src/views/pager/pager_example.js
@@ -30,7 +30,7 @@ export default props => (
       }]}
     >
       <GuideText>
-        Use the Pager in a tool bar
+        Use the Pager in a tool bar.
       </GuideText>
 
       <GuideDemo>
@@ -49,7 +49,7 @@ export default props => (
       }]}
     >
       <GuideText>
-        Use the Pager Buttons to navigate through a set of items
+        Use the Pager Buttons to navigate through a set of items.
       </GuideText>
 
       <GuideDemo>