R: Remove redundant useEffects
This commit is contained in:
parent
1b62c8fbc2
commit
483fc93b58
|
@ -1,6 +1,6 @@
|
||||||
'use client';
|
'use client';
|
||||||
|
|
||||||
import { useEffect } from 'react';
|
import { useLayoutEffect } from 'react';
|
||||||
import { TransformComponent, TransformWrapper } from 'react-zoom-pan-pinch';
|
import { TransformComponent, TransformWrapper } from 'react-zoom-pan-pinch';
|
||||||
|
|
||||||
import { useAppLayoutStore, useFitHeight } from '@/stores/appLayout';
|
import { useAppLayoutStore, useFitHeight } from '@/stores/appLayout';
|
||||||
|
@ -10,7 +10,7 @@ export function Component() {
|
||||||
const hideFooter = useAppLayoutStore(state => state.hideFooter);
|
const hideFooter = useAppLayoutStore(state => state.hideFooter);
|
||||||
const panelHeight = useFitHeight('0px');
|
const panelHeight = useFitHeight('0px');
|
||||||
|
|
||||||
useEffect(() => {
|
useLayoutEffect(() => {
|
||||||
hideFooter(true);
|
hideFooter(true);
|
||||||
return () => hideFooter(false);
|
return () => hideFooter(false);
|
||||||
}, [hideFooter]);
|
}, [hideFooter]);
|
||||||
|
|
|
@ -1,28 +1,17 @@
|
||||||
import { useEffect } from 'react';
|
|
||||||
|
|
||||||
import { urls, useConceptNavigation } from '@/app';
|
import { urls, useConceptNavigation } from '@/app';
|
||||||
import { useAuthSuspense } from '@/features/auth';
|
import { useAuthSuspense } from '@/features/auth';
|
||||||
|
|
||||||
import { Loader } from '@/components/Loader';
|
|
||||||
import { PARAMETER } from '@/utils/constants';
|
|
||||||
|
|
||||||
function HomePage() {
|
function HomePage() {
|
||||||
const router = useConceptNavigation();
|
const router = useConceptNavigation();
|
||||||
const { isAnonymous } = useAuthSuspense();
|
const { isAnonymous } = useAuthSuspense();
|
||||||
|
|
||||||
useEffect(() => {
|
if (isAnonymous) {
|
||||||
if (isAnonymous) {
|
router.replace(urls.manuals);
|
||||||
setTimeout(() => {
|
} else {
|
||||||
router.replace(urls.manuals);
|
router.replace(urls.library);
|
||||||
}, PARAMETER.refreshTimeout);
|
}
|
||||||
} else {
|
|
||||||
setTimeout(() => {
|
|
||||||
router.replace(urls.library);
|
|
||||||
}, PARAMETER.refreshTimeout);
|
|
||||||
}
|
|
||||||
}, [router, isAnonymous]);
|
|
||||||
|
|
||||||
return <Loader />;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
||||||
export default HomePage;
|
export default HomePage;
|
||||||
|
|
|
@ -1,6 +1,6 @@
|
||||||
'use client';
|
'use client';
|
||||||
|
|
||||||
import { useEffect } from 'react';
|
import { useLayoutEffect } from 'react';
|
||||||
import clsx from 'clsx';
|
import clsx from 'clsx';
|
||||||
|
|
||||||
import { useConceptNavigation } from '@/app/Navigation/NavigationContext';
|
import { useConceptNavigation } from '@/app/Navigation/NavigationContext';
|
||||||
|
@ -24,7 +24,7 @@ function OssTabs({ activeTab }: OssTabsProps) {
|
||||||
|
|
||||||
const hideFooter = useAppLayoutStore(state => state.hideFooter);
|
const hideFooter = useAppLayoutStore(state => state.hideFooter);
|
||||||
|
|
||||||
useEffect(() => {
|
useLayoutEffect(() => {
|
||||||
const oldTitle = document.title;
|
const oldTitle = document.title;
|
||||||
document.title = schema.title;
|
document.title = schema.title;
|
||||||
return () => {
|
return () => {
|
||||||
|
@ -32,7 +32,7 @@ function OssTabs({ activeTab }: OssTabsProps) {
|
||||||
};
|
};
|
||||||
}, [schema.title]);
|
}, [schema.title]);
|
||||||
|
|
||||||
useEffect(() => {
|
useLayoutEffect(() => {
|
||||||
hideFooter(activeTab === OssTabID.GRAPH);
|
hideFooter(activeTab === OssTabID.GRAPH);
|
||||||
}, [activeTab, hideFooter]);
|
}, [activeTab, hideFooter]);
|
||||||
|
|
||||||
|
|
|
@ -1,6 +1,6 @@
|
||||||
'use client';
|
'use client';
|
||||||
|
|
||||||
import { useEffect } from 'react';
|
import { useLayoutEffect } from 'react';
|
||||||
import clsx from 'clsx';
|
import clsx from 'clsx';
|
||||||
|
|
||||||
import { useConceptNavigation } from '@/app/Navigation/NavigationContext';
|
import { useConceptNavigation } from '@/app/Navigation/NavigationContext';
|
||||||
|
@ -31,7 +31,7 @@ function RSTabs({ activeID, activeTab }: RSTabsProps) {
|
||||||
const { setIsModified } = useModificationStore();
|
const { setIsModified } = useModificationStore();
|
||||||
const { schema, selected, setSelected, navigateRSForm } = useRSEdit();
|
const { schema, selected, setSelected, navigateRSForm } = useRSEdit();
|
||||||
|
|
||||||
useEffect(() => {
|
useLayoutEffect(() => {
|
||||||
const oldTitle = document.title;
|
const oldTitle = document.title;
|
||||||
document.title = schema.title;
|
document.title = schema.title;
|
||||||
return () => {
|
return () => {
|
||||||
|
@ -39,7 +39,7 @@ function RSTabs({ activeID, activeTab }: RSTabsProps) {
|
||||||
};
|
};
|
||||||
}, [schema.title]);
|
}, [schema.title]);
|
||||||
|
|
||||||
useEffect(() => {
|
useLayoutEffect(() => {
|
||||||
hideFooter(activeTab !== RSTabID.CARD);
|
hideFooter(activeTab !== RSTabID.CARD);
|
||||||
setIsModified(false);
|
setIsModified(false);
|
||||||
if (activeTab === RSTabID.CST_EDIT) {
|
if (activeTab === RSTabID.CST_EDIT) {
|
||||||
|
|
Loading…
Reference in New Issue
Block a user