Merge pull request #361 from samvera-labs/scale-section-btns

Fix scale for section buttons, to match with other player icons
samvera-labs · Jan 25, 2024 · f03ae28 · f03ae28
2 parents 8a08b76 + 082c2f2
commit f03ae28
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/components/MediaPlayer/VideoJS/components/js/VideoJSNextButton.js b/src/components/MediaPlayer/VideoJS/components/js/VideoJSNextButton.js
@@ -5,10 +5,10 @@ import '../styles/VideoJSSectionButtons.scss';
 
 const vjsComponent = videojs.getComponent('Component');
 
-const NextButtonIcon = ({ scale }) => {
+const NextButtonIcon = () => {
   return (
     <svg viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg" transform="rotate(0)"
-      style={{ fill: 'white', height: '1.25rem', width: '1.25rem', scale: scale }}>
+      style={{ fill: 'white', height: '1.25rem', width: '1.25rem' }}>
       <g strokeWidth="0" strokeLinecap="round" strokeLinejoin="round">
         <path d="M4 20L15.3333 12L4 4V20Z" fill="#ffffff"></path>
         <path d="M20 4H17.3333V20H20V4Z" fill="#ffffff"></path>
@@ -92,7 +92,7 @@ function NextButton({
         title={"Next"}
         onClick={() => handleNextClick(false)}
         onKeyDown={handleNextKeyDown}>
-        <NextButtonIcon scale="0.9" />
+        <NextButtonIcon />
       </button>
     </div >
   );

diff --git a/src/components/MediaPlayer/VideoJS/components/js/VideoJSPreviousButton.js b/src/components/MediaPlayer/VideoJS/components/js/VideoJSPreviousButton.js
@@ -5,10 +5,10 @@ import '../styles/VideoJSSectionButtons.scss';
 
 const vjsComponent = videojs.getComponent('Component');
 
-const PreviousButtonIcon = ({ scale }) => {
+const PreviousButtonIcon = () => {
   return (
     <svg viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg"
-      style={{ fill: 'white', height: '1.25rem', width: '1.25rem', scale: scale }}>
+      style={{ fill: 'white', height: '1.25rem', width: '1.25rem' }}>
       <g strokeWidth="0" strokeLinecap="round" strokeLinejoin="round">
         <path d="M20 4L8.66667 12L20 20V4Z" fill="#ffffff"></path>
         <path d="M4 20H6.66667V4H4V20Z" fill="#ffffff"></path>
@@ -94,7 +94,7 @@ function PreviousButton({
         title={canvasIndex == 0 ? "Replay" : "Previous"}
         onClick={() => handlePreviousClick(false)}
         onKeyDown={handlePreviousKeyDown}>
-        <PreviousButtonIcon scale="0.9" />
+        <PreviousButtonIcon />
       </button>
     </div>
   );