From f546e22ae166a06e58a34d43bea9a1d118c8e48f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?H=C3=A5var=20Aamb=C3=B8=20Fosstveit?= <havar@fosstveit.net>
Date: Fri, 22 May 2020 20:42:22 +0200
Subject: [PATCH] Remove video frame rate and screen sharing resolution, these
 are probably not required. Still configurable in app config. Fixes #415

---
 app/src/components/Settings/MediaSettings.js | 114 ++++++++++---------
 1 file changed, 58 insertions(+), 56 deletions(-)

diff --git a/app/src/components/Settings/MediaSettings.js b/app/src/components/Settings/MediaSettings.js
index a4e057f..6d65ff8 100644
--- a/app/src/components/Settings/MediaSettings.js
+++ b/app/src/components/Settings/MediaSettings.js
@@ -239,62 +239,64 @@ const MediaSettings = ({
 								/>
 							</FormHelperText>
 						</FormControl>
-						<FormControl className={classes.formControl}>
-							<Select
-								value={settings.frameRate || ''}
-								onChange={(event) =>
-								{
-									if (event.target.value)
-										roomClient.updateWebcam({ newFrameRate: event.target.value });
-								}}
-								name='Frame rate'
-								autoWidth
-								className={classes.selectEmpty}
-							>
-								{ [ 1, 5, 10, 15, 20, 25, 30 ].map((frameRate) =>
-								{
-									return (
-										<MenuItem key={frameRate} value={frameRate}>
-											{frameRate}
-										</MenuItem>
-									);
-								})}
-							</Select>
-							<FormHelperText>
-								<FormattedMessage
-									id='settings.frameRate'
-									defaultMessage='Select your video frame rate'
-								/>
-							</FormHelperText>
-						</FormControl>
-						<FormControl className={classes.formControl}>
-							<Select
-								value={settings.screenSharingResolution || ''}
-								onChange={(event) =>
-								{
-									if (event.target.value)
-										roomClient.updateScreenSharing({ newResolution: event.target.value });
-								}}
-								name='Screen sharing resolution'
-								autoWidth
-								className={classes.selectEmpty}
-							>
-								{resolutions.map((resolution, index) =>
-								{
-									return (
-										<MenuItem key={index} value={resolution.value}>
-											{resolution.label}
-										</MenuItem>
-									);
-								})}
-							</Select>
-							<FormHelperText>
-								<FormattedMessage
-									id='settings.screenSharingResolution'
-									defaultMessage='Select your screen sharing resolution'
-								/>
-							</FormHelperText>
-						</FormControl>
+						{ /*
+							<FormControl className={classes.formControl}>
+								<Select
+									value={settings.frameRate || ''}
+									onChange={(event) =>
+									{
+										if (event.target.value)
+											roomClient.updateWebcam({ newFrameRate: event.target.value });
+									}}
+									name='Frame rate'
+									autoWidth
+									className={classes.selectEmpty}
+								>
+									{ [ 1, 5, 10, 15, 20, 25, 30 ].map((frameRate) =>
+									{
+										return (
+											<MenuItem key={frameRate} value={frameRate}>
+												{frameRate}
+											</MenuItem>
+										);
+									})}
+								</Select>
+								<FormHelperText>
+									<FormattedMessage
+										id='settings.frameRate'
+										defaultMessage='Select your video frame rate'
+									/>
+								</FormHelperText>
+							</FormControl>
+							<FormControl className={classes.formControl}>
+								<Select
+									value={settings.screenSharingResolution || ''}
+									onChange={(event) =>
+									{
+										if (event.target.value)
+											roomClient.updateScreenSharing({ newResolution: event.target.value });
+									}}
+									name='Screen sharing resolution'
+									autoWidth
+									className={classes.selectEmpty}
+								>
+									{resolutions.map((resolution, index) =>
+									{
+										return (
+											<MenuItem key={index} value={resolution.value}>
+												{resolution.label}
+											</MenuItem>
+										);
+									})}
+								</Select>
+								<FormHelperText>
+									<FormattedMessage
+										id='settings.screenSharingResolution'
+										defaultMessage='Select your screen sharing resolution'
+									/>
+								</FormHelperText>
+							</FormControl>
+						*/ }
 						<FormControl className={classes.formControl}>
 							<Select
 								value={settings.screenSharingFrameRate || ''}