Browse Source

fix types and clean code

Yuki Takei 7 months ago
parent
commit
f8e2bfaf00

+ 4 - 20
apps/app/src/pages/[[...path]]/index.page.tsx

@@ -24,14 +24,11 @@ import {
 } from '~/states/page';
 } from '~/states/page';
 import { useHydratePageAtoms } from '~/states/page/hydrate';
 import { useHydratePageAtoms } from '~/states/page/hydrate';
 import { useRedirectFrom } from '~/states/page/redirect';
 import { useRedirectFrom } from '~/states/page/redirect';
-import {
-  useDisableLinkSharing,
-  useRendererConfig,
-} from '~/states/server-configurations';
+import { useRendererConfig } from '~/states/server-configurations';
 import { useHydrateServerConfigurationAtoms } from '~/states/server-configurations/hydrate';
 import { useHydrateServerConfigurationAtoms } from '~/states/server-configurations/hydrate';
-import { useHydrateSidebarAtoms } from '~/states/ui/sidebar/hydrate';
 import { useSetupGlobalSocket, useSetupGlobalSocketForPage } from '~/states/socket-io';
 import { useSetupGlobalSocket, useSetupGlobalSocketForPage } from '~/states/socket-io';
 import { useEditingMarkdown } from '~/states/ui/editor';
 import { useEditingMarkdown } from '~/states/ui/editor';
+import { useHydrateSidebarAtoms } from '~/states/ui/sidebar/hydrate';
 import { useSWRMUTxCurrentPageYjsData } from '~/stores/yjs';
 import { useSWRMUTxCurrentPageYjsData } from '~/stores/yjs';
 
 
 import type { NextPageWithLayout } from '../_app.page';
 import type { NextPageWithLayout } from '../_app.page';
@@ -55,7 +52,7 @@ declare global {
   var globalEmitter: EventEmitter;
   var globalEmitter: EventEmitter;
 }
 }
 
 
-const GrowiContextualSubNavigationSubstance = dynamic(() => import('~/client/components/Navbar/GrowiContextualSubNavigation'), { ssr: false });
+const GrowiContextualSubNavigation = dynamic(() => import('~/client/components/Navbar/GrowiContextualSubNavigation'), { ssr: false });
 
 
 const GrowiPluginsActivator = dynamic(() => import('~/features/growi-plugin/client/components').then(mod => mod.GrowiPluginsActivator), { ssr: false });
 const GrowiPluginsActivator = dynamic(() => import('~/features/growi-plugin/client/components').then(mod => mod.GrowiPluginsActivator), { ssr: false });
 
 
@@ -76,18 +73,6 @@ const ConflictDiffModal = dynamic(() => import('~/client/components/PageEditor/C
 
 
 const EditablePageEffects = dynamic(() => import('~/client/components/Page/EditablePageEffects').then(mod => mod.EditablePageEffects), { ssr: false });
 const EditablePageEffects = dynamic(() => import('~/client/components/Page/EditablePageEffects').then(mod => mod.EditablePageEffects), { ssr: false });
 
 
-// GrowiContextualSubNavigation for NOT shared page
-type GrowiContextualSubNavigationProps = {
-  isLinkSharingDisabled: boolean,
-}
-
-const GrowiContextualSubNavigation = (props: GrowiContextualSubNavigationProps): JSX.Element => {
-  const { isLinkSharingDisabled } = props;
-  const [currentPage] = useCurrentPageData();
-  return (
-    <GrowiContextualSubNavigationSubstance currentPage={currentPage} isLinkSharingDisabled={isLinkSharingDisabled} />
-  );
-};
 
 
 const isInitialProps = (props: Props): props is (InitialProps & SameRouteEachProps) => {
 const isInitialProps = (props: Props): props is (InitialProps & SameRouteEachProps) => {
   return 'isNextjsRoutingTypeInitial' in props && props.isNextjsRoutingTypeInitial;
   return 'isNextjsRoutingTypeInitial' in props && props.isNextjsRoutingTypeInitial;
@@ -109,7 +94,6 @@ const Page: NextPageWithLayout<Props> = (props: Props) => {
   const [currentPagePath] = useCurrentPagePath();
   const [currentPagePath] = useCurrentPagePath();
   const [isNotFound] = usePageNotFound();
   const [isNotFound] = usePageNotFound();
   const [rendererConfig] = useRendererConfig();
   const [rendererConfig] = useRendererConfig();
-  const [disableLinkSharing] = useDisableLinkSharing();
   const [, setRedirectFrom] = useRedirectFrom();
   const [, setRedirectFrom] = useRedirectFrom();
   const [, setIsSharedUser] = useIsSharedUser();
   const [, setIsSharedUser] = useIsSharedUser();
   const [, setIsSearchPage] = useIsSearchPage();
   const [, setIsSearchPage] = useIsSearchPage();
@@ -163,7 +147,7 @@ const Page: NextPageWithLayout<Props> = (props: Props) => {
       </Head>
       </Head>
       <div className="dynamic-layout-root justify-content-between">
       <div className="dynamic-layout-root justify-content-between">
 
 
-        <GrowiContextualSubNavigation isLinkSharingDisabled={disableLinkSharing} />
+        <GrowiContextualSubNavigation currentPage={currentPage} />
 
 
         <PageView
         <PageView
           className="d-edit-none"
           className="d-edit-none"

+ 1 - 1
apps/app/src/pages/common-props/index.ts

@@ -3,4 +3,4 @@ export {
   type CommonEachProps, getServerSideCommonEachProps,
   type CommonEachProps, getServerSideCommonEachProps,
 } from './commons';
 } from './commons';
 export { getServerSideI18nProps } from './i18n';
 export { getServerSideI18nProps } from './i18n';
-export { getServerSideUserUISettingsProps } from './user-ui-settings';
+export { getServerSideUserUISettingsProps, type UserUISettingsProps } from './user-ui-settings';

+ 1 - 2
apps/app/src/pages/general-page/types.ts

@@ -4,11 +4,10 @@ import type {
 
 
 import type { RendererConfig } from '~/interfaces/services/renderer';
 import type { RendererConfig } from '~/interfaces/services/renderer';
 import type { ISidebarConfig } from '~/interfaces/sidebar-config';
 import type { ISidebarConfig } from '~/interfaces/sidebar-config';
-import type { CommonEachProps, CommonInitialProps } from '~/pages/utils/commons';
-import type { UserUISettingsProps } from '~/pages/utils/user-ui-settings';
 import type { PageDocument } from '~/server/models/page';
 import type { PageDocument } from '~/server/models/page';
 import type { ServerConfigurationHyderateArgs } from '~/states/server-configurations/hydrate';
 import type { ServerConfigurationHyderateArgs } from '~/states/server-configurations/hydrate';
 
 
+import type { CommonEachProps, CommonInitialProps, UserUISettingsProps } from '../common-props';
 
 
 export type IPageToShowRevisionWithMeta = IDataWithMeta<IPagePopulatedToShowRevision & PageDocument, IPageInfo>;
 export type IPageToShowRevisionWithMeta = IDataWithMeta<IPagePopulatedToShowRevision & PageDocument, IPageInfo>;