LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Move MAIN_CLASS to constants file for better code-splitting.

Browse source 
Cross-importing was making things hard to code-split efficiently, because the entire file gets evaluated when imported.
This commit is contained in:
infinite-persistence 2021-12-10 12:28:37 +08:00 committed by infinite-persistence
parent 0d59ce4f8c
commit 9307511c88
3 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
ui/component/claimList/view.jsx
View file

@ -1,5 +1,5 @@
// @flow
import { MAIN_CLASS } from 'component/page/view';
import { MAIN_CLASS } from 'constants/classnames';
import type { Node } from 'react';
import React, { useEffect } from 'react';
import classnames from 'classnames';

4
ui/component/page/view.jsx
View file

@ -2,6 +2,7 @@
import type { Node } from 'react';
import React, { Fragment } from 'react';
import classnames from 'classnames';
import { MAIN_CLASS } from 'constants/classnames';
import { lazyImport } from 'util/lazyImport';
import SideNavigation from 'component/sideNavigation';
import SettingsSideNavigation from 'component/settingsSideNavigation';
@ -16,12 +17,9 @@ import { parseURI } from 'util/lbryURI';
const Footer = lazyImport(() => import('web/component/footer' /* webpackChunkName: "footer" */));
export const MAIN_CLASS = 'main';
type Props = {
children: Node | Array<Node>,
className: ?string,
autoUpdateDownloaded: boolean,
isUpgradeAvailable: boolean,
authPage: boolean,
filePage: boolean,
settingsPage?: boolean,

1
ui/constants/classnames.js
View file

@ -1 +1,2 @@
export const MAIN_CLASS = 'main';
export const COMMENT_HIGHLIGHTED = 'comment--highlighted';