Apply suggestions from code review

Co-authored-by: Jan Keromnes <[email protected]>

Author: mustard-mh

components/dashboard/src/settings/Preferences.tsx

@@ -34,7 +34,7 @@ export default function Preferences() {
     const { setIsDark, isDark } = useContext(ThemeContext);
 
     const updateUserIDEInfo = async (defaultDesktopIde: string, defaultIde: string) => {
-        let useDesktopIde = defaultDesktopIde !== DesktopNoneId;
+        const useDesktopIde = defaultDesktopIde !== DesktopNoneId;
         const additionalData = user?.additionalData ?? {};
         const settings = additionalData.ideSettings ?? {};
         settings.useDesktopIde = useDesktopIde;
@@ -116,7 +116,7 @@ export default function Preferences() {
                             })
                         }
                     </div>
-                    {ideOptions.options[defaultIde] && ideOptions.options[defaultIde].notes &&
+                    {ideOptions.options[defaultIde]?.notes &&
                         <InfoBox className="my-5 max-w-2xl"><ul>
                             {ideOptions.options[defaultIde].notes?.map((x, idx) => <li className={idx > 0 ? "mt-2" : ""}>{x}</li>)}
                         </ul></InfoBox>
@@ -131,14 +131,13 @@ export default function Preferences() {
                                 let selected = defaultDesktopIde === id;
                                 const onSelect = () => actuallySetDefaultDesktopIde(id);
                                 if (id === DesktopNoneId) {
-                                    selected = defaultDesktopIde === DesktopNoneId
                                     option.logo = isDark ? DesktopNoneLogoDark : DesktopNoneLogoLight
                                 }
                                 return renderIdeOption(option, selected, onSelect);
                             })
                         }
                     </div>
-                    {ideOptions.options[defaultDesktopIde] && ideOptions.options[defaultDesktopIde].notes &&
+                    {ideOptions.options[defaultDesktopIde]?.notes &&
                         <InfoBox className="my-5 max-w-2xl"><ul>
                             {ideOptions.options[defaultDesktopIde].notes?.map((x, idx) => <li className={idx > 0 ? "mt-2" : ""}>{x}</li>)}
                         </ul></InfoBox>