From 69f5abb04029e74e2311e6f6986197d3835924e3 Mon Sep 17 00:00:00 2001
From: David Joy <djoy@edx.org>
Date: Fri, 10 Jan 2020 13:00:27 -0500
Subject: [PATCH] Hooking up the unit buttons.

---
 .../sub-section/SubSectionNavigation.jsx      | 24 +++++++++++++++----
 1 file changed, 19 insertions(+), 5 deletions(-)

diff --git a/src/learning-sequence/sub-section/SubSectionNavigation.jsx b/src/learning-sequence/sub-section/SubSectionNavigation.jsx
index 58db8925..3000c4f6 100644
--- a/src/learning-sequence/sub-section/SubSectionNavigation.jsx
+++ b/src/learning-sequence/sub-section/SubSectionNavigation.jsx
@@ -48,6 +48,10 @@ export default function SubSectionNavigation() {
     }
   });
 
+  const handleUnitClick = useCallback((unit) => {
+    history.push(`/course/${courseId}/${unit.parentId}/${unit.id}`);
+  });
+
   return (
     <nav className="flex-grow-0 d-flex w-100 mb-3">
       <Button
@@ -57,7 +61,7 @@ export default function SubSectionNavigation() {
       >
         Previous
       </Button>
-      <UnitNavigation />
+      <UnitNavigation clickHandler={handleUnitClick} />
       <Button
         key="next"
         className="btn-outline-primary"
@@ -69,25 +73,34 @@ export default function SubSectionNavigation() {
   );
 }
 
-function UnitNavigation() {
+function UnitNavigation({ clickHandler }) {
   const units = useCurrentSubSectionUnits();
   const currentUnit = useCurrentUnit();
 
   return (
     <div className="btn-group ml-2 mr-2 flex-grow-1 d-flex" role="group">
       {units.map(unit => (
-        <UnitButton key={unit.id} unit={unit} disabled={unit.id === currentUnit.id} />
+        <UnitButton key={unit.id} unit={unit} disabled={unit.id === currentUnit.id} clickHandler={clickHandler} />
       ))}
     </div>
   );
 }
 
-function UnitButton({ unit: { id, type }, disabled }) {
+UnitNavigation.propTypes = {
+  clickHandler: PropTypes.func.isRequired,
+};
+
+function UnitButton({ unit, disabled, clickHandler }) {
+  const { id, type } = unit;
+  const handleClick = useCallback(() => {
+    clickHandler(unit);
+  }, [unit]);
+
   return (
     <Button
       key={id}
       className="btn-outline-secondary unit-button flex-grow-1"
-      onClick={() => console.log(id)}
+      onClick={handleClick}
       disabled={disabled}
     >
       <UnitIcon type={type} />
@@ -101,4 +114,5 @@ UnitButton.propTypes = {
     type: PropTypes.oneOf(['video', 'other', 'vertical', 'problem']).isRequired,
   }).isRequired,
   disabled: PropTypes.bool.isRequired,
+  clickHandler: PropTypes.func.isRequired,
 };