From 16e6a3cf4dda63f156228a135f1ac541efed07b6 Mon Sep 17 00:00:00 2001
From: Tim <tymmesyde@gmail.com>
Date: Wed, 19 Mar 2025 19:23:40 +0100
Subject: [PATCH] refactor: remove unecessary props

---
 .../NavBar/HorizontalNavBar/HorizontalNavBar.js          | 7 +++----
 src/routes/Player/ControlBar/ControlBar.js               | 8 +-------
 src/routes/Player/OptionsMenu/OptionsMenu.js             | 9 ++-------
 src/routes/Player/SideDrawerButton/SideDrawerButton.tsx  | 9 +++------
 4 files changed, 9 insertions(+), 24 deletions(-)

diff --git a/src/components/NavBar/HorizontalNavBar/HorizontalNavBar.js b/src/components/NavBar/HorizontalNavBar/HorizontalNavBar.js
index 9f741c603..0cedd8638 100644
--- a/src/components/NavBar/HorizontalNavBar/HorizontalNavBar.js
+++ b/src/components/NavBar/HorizontalNavBar/HorizontalNavBar.js
@@ -12,7 +12,7 @@ const NavMenu = require('./NavMenu');
 const styles = require('./styles');
 const { t } = require('i18next');
 
-const HorizontalNavBar = React.memo(({ className, route, query, title, backButton, searchBar, fullscreenButton, navMenu, onContextMenu, ...props }) => {
+const HorizontalNavBar = React.memo(({ className, route, query, title, backButton, searchBar, fullscreenButton, navMenu, ...props }) => {
     const backButtonOnClick = React.useCallback(() => {
         window.history.back();
     }, []);
@@ -25,7 +25,7 @@ const HorizontalNavBar = React.memo(({ className, route, query, title, backButto
         </Button>
     ), []);
     return (
-        <nav {...props} className={classnames(className, styles['horizontal-nav-bar-container'])} onContextMenu={onContextMenu}>
+        <nav {...props} className={classnames(className, styles['horizontal-nav-bar-container'])}>
             {
                 backButton ?
                     <Button className={classnames(styles['button-container'], styles['back-button-container'])} tabIndex={-1} onClick={backButtonOnClick}>
@@ -82,8 +82,7 @@ HorizontalNavBar.propTypes = {
     backButton: PropTypes.bool,
     searchBar: PropTypes.bool,
     fullscreenButton: PropTypes.bool,
-    navMenu: PropTypes.bool,
-    onContextMenu: PropTypes.func,
+    navMenu: PropTypes.bool
 };
 
 module.exports = HorizontalNavBar;
diff --git a/src/routes/Player/ControlBar/ControlBar.js b/src/routes/Player/ControlBar/ControlBar.js
index c81d0e38e..745e2bd49 100644
--- a/src/routes/Player/ControlBar/ControlBar.js
+++ b/src/routes/Player/ControlBar/ControlBar.js
@@ -40,9 +40,6 @@ const ControlBar = ({
     onToggleSideDrawer,
     onToggleOptionsMenu,
     onToggleStatisticsMenu,
-    onContextMenu,
-    onTouchStart,
-    onTouchEnd,
     ...props
 }) => {
     const { chromecast } = useServices();
@@ -106,7 +103,7 @@ const ControlBar = ({
         };
     }, []);
     return (
-        <div {...props} className={classnames(className, styles['control-bar-container'])} onContextMenu={onContextMenu} onTouchStart={onTouchStart} onTouchEnd={onTouchEnd}>
+        <div {...props} className={classnames(className, styles['control-bar-container'])}>
             <SeekBar
                 className={styles['seek-bar']}
                 time={time}
@@ -209,9 +206,6 @@ ControlBar.propTypes = {
     onToggleSideDrawer: PropTypes.func,
     onToggleOptionsMenu: PropTypes.func,
     onToggleStatisticsMenu: PropTypes.func,
-    onContextMenu: PropTypes.func,
-    onTouchStart: PropTypes.func,
-    onTouchEnd: PropTypes.func,
 };
 
 module.exports = ControlBar;
diff --git a/src/routes/Player/OptionsMenu/OptionsMenu.js b/src/routes/Player/OptionsMenu/OptionsMenu.js
index 0065d9b55..56241e009 100644
--- a/src/routes/Player/OptionsMenu/OptionsMenu.js
+++ b/src/routes/Player/OptionsMenu/OptionsMenu.js
@@ -9,7 +9,7 @@ const { useServices } = require('stremio/services');
 const Option = require('./Option');
 const styles = require('./styles');
 
-const OptionsMenu = ({ className, stream, playbackDevices, style }) => {
+const OptionsMenu = ({ className, stream, playbackDevices }) => {
     const { t } = useTranslation();
     const { core } = useServices();
     const platform = usePlatform();
@@ -71,7 +71,7 @@ const OptionsMenu = ({ className, stream, playbackDevices, style }) => {
     }, []);
 
     return (
-        <div style={style} className={classnames(className, styles['options-menu-container'])} onMouseDown={onMouseDown}>
+        <div className={classnames(className, styles['options-menu-container'])} onMouseDown={onMouseDown}>
             {
                 streamingUrl || downloadUrl ?
                     <Option
@@ -111,14 +111,9 @@ const OptionsMenu = ({ className, stream, playbackDevices, style }) => {
 };
 
 OptionsMenu.propTypes = {
-    menuRef: PropTypes.oneOfType([
-        PropTypes.func,
-        PropTypes.shape({ current: PropTypes.any })
-    ]),
     className: PropTypes.string,
     stream: PropTypes.object,
     playbackDevices: PropTypes.array,
-    style: PropTypes.object,
 };
 
 module.exports = OptionsMenu;
diff --git a/src/routes/Player/SideDrawerButton/SideDrawerButton.tsx b/src/routes/Player/SideDrawerButton/SideDrawerButton.tsx
index 9d5897548..81409f434 100644
--- a/src/routes/Player/SideDrawerButton/SideDrawerButton.tsx
+++ b/src/routes/Player/SideDrawerButton/SideDrawerButton.tsx
@@ -1,6 +1,6 @@
 // Copyright (C) 2017-2024 Smart code 203358507
 
-import React, { BaseSyntheticEvent } from 'react';
+import React from 'react';
 import classNames from 'classnames';
 import Icon from '@stremio/stremio-icons/react';
 import styles from './SideDrawerButton.less';
@@ -8,14 +8,11 @@ import styles from './SideDrawerButton.less';
 type Props = {
     className: string,
     onClick: () => void,
-    onContextMenu: () => void,
-    onTouchStart: (event: BaseSyntheticEvent) => void,
-    onTouchEnd: () => void,
 };
 
-const SideDrawerButton = ({ className, onClick, onContextMenu, onTouchStart, onTouchEnd }: Props) => {
+const SideDrawerButton = ({ className, onClick }: Props) => {
     return (
-        <div className={classNames(className, styles['side-drawer-button'])} onClick={onClick} onContextMenu={onContextMenu} onTouchStart={onTouchStart} onTouchEnd={onTouchEnd}>
+        <div className={classNames(className, styles['side-drawer-button'])} onClick={onClick}>
             <Icon name={'chevron-back'} className={styles['icon']} />
         </div>
     );