Repository: FormidableLabs/spectacle (ts)

Issue:
Fullscreen button is not applying the tertiary color from the theme
Hey!

Im noticing a bug where the fullscreen button is not applying the tertiary color i set in the theme. This happens both when hovered and not hovered.

Pull request:
Fullscreen Button Takes Applied Color
This updates how color is passed to the fullscreen button - if no color is specified in a theme, it remains white, or default, but otherwise it should take on the color specified. Fixes #563 

Patch:
diff --git a/src/components/fullscreen-button.js b/src/components/fullscreen-button.js
index b96394a..de4355d 100644
--- a/src/components/fullscreen-button.js
+++ b/src/components/fullscreen-button.js
@@ -23,7 +23,9 @@ const FullscreenButton = props => (
       <path d="M0 0h24v24H0z" fill="none" />
       <path
         d="M7 14H5v5h5v-2H7v-3zm-2-4h2V7h3V5H5v5zm12 7h-3v2h5v-5h-2v3zM14 5v2h3v3h2V5h-5z"
-        fill="currentColor"
+        fill={
+          props.styles && props.styles.fill ? props.styles.fill : 'currentColor'
+        }
       />
     </svg>
   </Button>
diff --git a/src/components/fullscreen.js b/src/components/fullscreen.js
index 03a8f82..b79b3bc 100644
--- a/src/components/fullscreen.js
+++ b/src/components/fullscreen.js
@@ -37,7 +37,7 @@ export class Fullscreen extends Component {
     return (
       <StyledFullscreen
         onClick={() => this.toggleFullscreen()}
-        style={this.context.styles.fullscreen}
+        styles={this.context.styles.fullscreen}
         viewBox="0 0 512 512"
       />
     );


Test patch:
diff --git a/src/components/__snapshots__/fullscreen.test.js.snap b/src/components/__snapshots__/fullscreen.test.js.snap
index ce0a75a..9671168 100644
--- a/src/components/__snapshots__/fullscreen.test.js.snap
+++ b/src/components/__snapshots__/fullscreen.test.js.snap
@@ -4,27 +4,26 @@ exports[`<Fullscreen /> should render correctly. 1`] = `
 <Fullscreen>
   <Styled(FullscreenButton)
     onClick={[Function]}
-    style={undefined}
+    styles={undefined}
     viewBox="0 0 512 512"
   >
     <FullscreenButton
       className="css-12n5ud e4v3eea0"
       onClick={[Function]}
-      style={undefined}
+      styles={undefined}
       viewBox="0 0 512 512"
     >
       <Styled(button)
         aria-label="Toggle full screen"
         className="css-12n5ud e4v3eea0"
         onClick={[Function]}
-        style={undefined}
+        styles={undefined}
         viewBox="0 0 512 512"
       >
         <button
           aria-label="Toggle full screen"
           className="e4v3eea0 css-1hv29sw ev51h780"
           onClick={[Function]}
-          style={undefined}
           viewBox="0 0 512 512"
         >
           <svg
diff --git a/src/components/__snapshots__/manager.test.js.snap b/src/components/__snapshots__/manager.test.js.snap
index 2884b20..030c556 100644
--- a/src/components/__snapshots__/manager.test.js.snap
+++ b/src/components/__snapshots__/manager.test.js.snap
@@ -335,27 +335,26 @@ exports[`<Manager /> should render correctly. 1`] = `
         <Fullscreen>
           <Styled(FullscreenButton)
             onClick={[Function]}
-            style={undefined}
+            styles={undefined}
             viewBox="0 0 512 512"
           >
             <FullscreenButton
               className="css-12n5ud e4v3eea0"
               onClick={[Function]}
-              style={undefined}
+              styles={undefined}
               viewBox="0 0 512 512"
             >
               <Styled(button)
                 aria-label="Toggle full screen"
                 className="css-12n5ud e4v3eea0"
                 onClick={[Function]}
-                style={undefined}
+                styles={undefined}
                 viewBox="0 0 512 512"
               >
                 <button
                   aria-label="Toggle full screen"
                   className="e4v3eea0 css-1hv29sw ev51h780"
                   onClick={[Function]}
-                  style={undefined}
                   viewBox="0 0 512 512"
                 >
                   <svg
@@ -669,27 +668,26 @@ exports[`<Manager /> should render the overview configuration when specified. 1`
         <Fullscreen>
           <Styled(FullscreenButton)
             onClick={[Function]}
-            style={undefined}
+            styles={undefined}
             viewBox="0 0 512 512"
           >
             <FullscreenButton
               className="css-12n5ud e4v3eea0"
               onClick={[Function]}
-              style={undefined}
+              styles={undefined}
               viewBox="0 0 512 512"
             >
               <Styled(button)
                 aria-label="Toggle full screen"
                 className="css-12n5ud e4v3eea0"
                 onClick={[Function]}
-                style={undefined}
+                styles={undefined}
                 viewBox="0 0 512 512"
               >
                 <button
                   aria-label="Toggle full screen"
                   className="e4v3eea0 css-1hv29sw ev51h780"
                   onClick={[Function]}
-                  style={undefined}
                   viewBox="0 0 512 512"
                 >
                   <svg
@@ -1093,27 +1091,26 @@ exports[`<Manager /> should render with slideset slides 1`] = `
         <Fullscreen>
           <Styled(FullscreenButton)
             onClick={[Function]}
-            style={undefined}
+            styles={undefined}
             viewBox="0 0 512 512"
           >
             <FullscreenButton
               className="css-12n5ud e4v3eea0"
               onClick={[Function]}
-              style={undefined}
+              styles={undefined}
               viewBox="0 0 512 512"
             >
               <Styled(button)
                 aria-label="Toggle full screen"
                 className="css-12n5ud e4v3eea0"
                 onClick={[Function]}
-                style={undefined}
+                styles={undefined}
                 viewBox="0 0 512 512"
               >
                 <button
                   aria-label="Toggle full screen"
                   className="e4v3eea0 css-1hv29sw ev51h780"
                   onClick={[Function]}
-                  style={undefined}
                   viewBox="0 0 512 512"
                 >
                   <svg