Merge pull request #131 from BetterSEQTA/main-cf-autofix

Apply fixes from CodeFactor

src/SEQTA.ts

@@ -336,7 +336,7 @@ async function DeleteWhatsNew() {
   if (!settingsState.animations) {
     bkelement?.remove()
     return
-  };
+  }
 
   animate(
     [popup, bkelement!],
@@ -2109,7 +2109,7 @@ export async function loadHomePage() {
         easing: [.22, .03, .26, 1]  
       }
     )
-  };
+  }
 
   callHomeTimetable(TodayFormatted)
 

src/css/injected.scss

@@ -469,7 +469,6 @@ ol:has(.MessageList__avatar___2wxyb svg) {
   border-bottom-color: transparent !important;
 }
 #main > .timetablepage > .quickbar.below::before {
-  border-bottom-color: transparent;
   top: -23px;
   background-color: inherit;
   clip-path: polygon(50% 40%, 0 100%, 100% 100%);
@@ -488,11 +487,10 @@ ol:has(.MessageList__avatar___2wxyb svg) {
   border-top-color: transparent;
 }
 #main > .timetablepage > .quickbar.above::before {
-  border-bottom-color: transparent;
+  border-bottom-color: transparent !important;
   bottom: -23px !important;
   background-color: inherit;
   clip-path: polygon(50% 40%, 0 0, 100% 0);
-  border-top-color: transparent !important;
 }
 #main .timetablepage .actions a,
 #main .timetablepage .actions button {
@@ -955,7 +953,7 @@ div > ol:has(.uiFileHandlerWrapper) {
 .timetable-container {
   width: 100%;
   overflow: visible !important;
-  border-radius: 0 16px 0 16px;
+  border-radius: 0 16px;
   z-index: 1;
   height: auto;
   background: var(--better-main);
@@ -2814,7 +2812,6 @@ li.MessageList__unread___3imtO {
 }
 .tooltip {
   display: inline-block;
-  background: transparent;
   z-index: 5 !important;
   width: 28px;
   background: none;
@@ -3049,4 +3046,4 @@ li[data-message] {
     transform: translateY(0);
     scale: 1;
   }
-}
+}

src/interface/SettingsContext.tsx

@@ -1,4 +1,4 @@
-import React, { createContext, useContext, useState, ReactNode } from 'react';
+import React, { createContext, ReactNode, useContext, useState } from 'react';
 import { SettingsState } from './types/AppProps';
 import useSettingsState from './hooks/settingsState';
 

src/interface/components/TabbedContainer.tsx

@@ -1,4 +1,4 @@
-import React, { useState, useRef, useEffect, memo } from 'react';
+import React, { memo, useEffect, useRef, useState } from 'react';
 import { motion } from 'framer-motion';
 import type { TabbedContainerProps } from '../types/TabbedContainerProps';
 

src/interface/components/ThemeSelector.tsx

@@ -1,5 +1,5 @@
-import React, { useEffect, useState, useCallback, forwardRef, useImperativeHandle, ForwardRefExoticComponent, RefAttributes } from 'react';
-import { listThemes, deleteTheme, setTheme, disableTheme, getDownloadedThemes, sendThemeUpdate } from '../hooks/ThemeManagment';
+import React, { forwardRef, ForwardRefExoticComponent, RefAttributes, useCallback, useEffect, useImperativeHandle, useState } from 'react';
+import { deleteTheme, disableTheme, getDownloadedThemes, listThemes, sendThemeUpdate, setTheme } from '../hooks/ThemeManagment';
 import { DeleteDownloadedTheme } from '../pages/Store';
 import { ThemeCover } from './ThemeCover';
 import browser from 'webextension-polyfill';

src/interface/components/useVisibility.tsx

@@ -1,4 +1,4 @@
-import { useState, useEffect, useRef } from 'react';
+import { useEffect, useRef, useState } from 'react';
 
 interface Options {
   root?: Element | null;

src/interface/main.tsx

@@ -1,6 +1,6 @@
 import React from 'react';
 import ReactDOM from 'react-dom/client';
-import { HashRouter, Routes, Route } from 'react-router-dom';
+import { HashRouter, Route, Routes } from 'react-router-dom';
 import { ErrorBoundary } from "react-error-boundary";
 
 import './index.css';

src/interface/pages/SettingsPage/Shortcuts.tsx

@@ -1,7 +1,7 @@
-import { useState, memo, useCallback } from "react";
+import { memo, useCallback, useState } from "react";
 import Switch from "../../components/Switch";
 import { useSettingsContext } from "../../SettingsContext";
-import { motion, AnimatePresence } from "framer-motion";
+import { AnimatePresence, motion } from "framer-motion";
 import { CustomShortcut } from "../../types/AppProps";
 
 function formatUrl(inputUrl: string) {

src/interface/pages/SettingsPage/Themes.tsx

@@ -1,4 +1,4 @@
-import { FC, createRef, useState } from 'react';
+import { createRef, FC, useState } from 'react';
 import BackgroundSelector from '../../components/BackgroundSelector';
 //import ThemeSelector from '../../components/ThemeSelector';
 import { memo } from 'react';

src/interface/pages/Store.tsx

@@ -2,7 +2,7 @@ import { useEffect, useRef, useState } from 'react';
 import { Swiper, SwiperSlide } from 'swiper/react';
 import Header from '../components/store/header';
 import { Autoplay } from 'swiper/modules';
-import { motion, AnimatePresence } from 'framer-motion';
+import { AnimatePresence, motion } from 'framer-motion';
 
 import 'swiper/css';
 import 'swiper/css/pagination';
@@ -110,7 +110,7 @@ const Store = () => {
     if (!themeContent) {
       alert('There was an error, The theme was not found!')
       return
-    };
+    }
 
     setInstallingThemes([...installingThemes, id]);
 
@@ -125,7 +125,7 @@ const Store = () => {
     if (!themeContent) {
       alert('There was an error, The theme was not found!')
       return
-    };
+    }
 
     setInstallingThemes([...installingThemes, id]);
 

src/seqta/ui/AddBetterSEQTAElements.ts

@@ -1,4 +1,4 @@
-import { GetThresholdOfColor, SendNewsPage, addExtensionSettings, enableAnimatedBackground, loadHomePage, setupSettingsButton } from "../../SEQTA";
+import { addExtensionSettings, enableAnimatedBackground, GetThresholdOfColor, loadHomePage, SendNewsPage, setupSettingsButton } from "../../SEQTA";
 import { updateBgDurations } from "./Animation";
 import { appendBackgroundToUI } from "./ImageBackgrounds";
 import stringToHTML from "../utils/stringToHTML";

src/seqta/ui/themes/UpdateThemePreview.ts

@@ -1,5 +1,5 @@
 import { CustomThemeBase64 } from '../../../interface/types/CustomThemes';
-import { imageData, removeImageFromDocument, UpdateImageData, applyCustomCSS } from './Themes';
+import { applyCustomCSS, imageData, removeImageFromDocument, UpdateImageData } from './Themes';
 import { settingsState } from '../../utils/listeners/SettingsState';
 
 

src/seqta/utils/listeners/MessageListener.ts

@@ -1,6 +1,6 @@
 import browser from 'webextension-polyfill'
 
-import { MenuOptionsOpen, OpenMenuOptions, OpenWhatsNewPopup, closeSettings } from '../../../SEQTA';
+import { closeSettings, MenuOptionsOpen, OpenMenuOptions, OpenWhatsNewPopup } from '../../../SEQTA';
 import { deleteTheme } from '../../ui/themes/deleteTheme';
 import { getAvailableThemes } from '../../ui/themes/getAvailableThemes';
 import { saveTheme } from '../../ui/themes/saveTheme';

src/seqta/utils/listeners/StorageChanges.ts

@@ -1,14 +1,14 @@
 import { settingsState } from './SettingsState';
 import { updateAllColors } from '../../ui/colors/Manager';
 import {
+  addShortcuts,
   CreateBackground,
   CreateCustomShortcutDiv,
+  disableNotificationCollector,
+  enableNotificationCollector,
   FilterUpcomingAssessments,
   RemoveBackground,
   RemoveShortcutDiv,
-  addShortcuts,
-  disableNotificationCollector,
-  enableNotificationCollector,
 } from '../../../SEQTA';
 import { updateBgDurations } from '../../ui/Animation';
 import browser from 'webextension-polyfill';