aboutsummaryrefslogtreecommitdiffstats
path: root/packages/react-shared/src/components
diff options
context:
space:
mode:
authorFabio Berger <me@fabioberger.com>2018-10-20 00:40:55 +0800
committerFabio Berger <me@fabioberger.com>2018-10-20 00:40:55 +0800
commit0de654bbd52f7d4702cec9f1a9a5a2cbb793181b (patch)
treeba341b86b4000cf94669a65d0fa2499c05897db9 /packages/react-shared/src/components
parentd129c922edd8bc0319273488b23a5257d736502d (diff)
downloaddexon-sol-tools-0de654bbd52f7d4702cec9f1a9a5a2cbb793181b.tar
dexon-sol-tools-0de654bbd52f7d4702cec9f1a9a5a2cbb793181b.tar.gz
dexon-sol-tools-0de654bbd52f7d4702cec9f1a9a5a2cbb793181b.tar.bz2
dexon-sol-tools-0de654bbd52f7d4702cec9f1a9a5a2cbb793181b.tar.lz
dexon-sol-tools-0de654bbd52f7d4702cec9f1a9a5a2cbb793181b.tar.xz
dexon-sol-tools-0de654bbd52f7d4702cec9f1a9a5a2cbb793181b.tar.zst
dexon-sol-tools-0de654bbd52f7d4702cec9f1a9a5a2cbb793181b.zip
fix: scroll lag on doc reference and wiki pages by using react-scroll `spy` and only updating the sidebar menu items whose active state had changed
Diffstat (limited to 'packages/react-shared/src/components')
-rw-r--r--packages/react-shared/src/components/link.tsx16
-rw-r--r--packages/react-shared/src/components/nested_sidebar_menu.tsx119
2 files changed, 15 insertions, 120 deletions
diff --git a/packages/react-shared/src/components/link.tsx b/packages/react-shared/src/components/link.tsx
index 5a456109b..089e6e2ba 100644
--- a/packages/react-shared/src/components/link.tsx
+++ b/packages/react-shared/src/components/link.tsx
@@ -7,7 +7,7 @@ import * as validUrl from 'valid-url';
import { LinkType } from '../types';
import { constants } from '../utils/constants';
-interface LinkProps {
+interface BaseLinkProps {
to: string;
shouldOpenInNewTab?: boolean;
className?: string;
@@ -18,6 +18,12 @@ interface LinkProps {
fontColor?: string;
}
+interface ScrollLinkProps extends BaseLinkProps {
+ onActivityChanged?: (isActive: boolean) => void;
+}
+
+type LinkProps = BaseLinkProps & ScrollLinkProps;
+
export interface LinkState {}
/**
@@ -103,11 +109,14 @@ export class Link extends React.Component<LinkProps, LinkState> {
<ScrollLink
to={this.props.to}
offset={0}
+ spy={true}
hashSpy={true}
duration={constants.DOCS_SCROLL_DURATION_MS}
containerId={constants.SCROLL_CONTAINER_ID}
className={this.props.className}
style={styleWithDefault}
+ onSetActive={this._onActivityChanged.bind(this, true)}
+ onSetInactive={this._onActivityChanged.bind(this, false)}
>
<span onClick={this._onClickPropagateClickEventAroundScrollLink.bind(this)}>
{this.props.children}
@@ -119,6 +128,11 @@ export class Link extends React.Component<LinkProps, LinkState> {
throw new Error(`Unrecognized LinkType: ${type}`);
}
}
+ private _onActivityChanged(isActive: boolean): void {
+ if (this.props.onActivityChanged) {
+ this.props.onActivityChanged(isActive);
+ }
+ }
// HACK(fabio): For some reason, the react-scroll link decided to stop the propagation of click events.
// We do however rely on these events being propagated in certain scenarios (e.g when the link
// is within a dropdown we want to close upon being clicked). Because of this, we register the
diff --git a/packages/react-shared/src/components/nested_sidebar_menu.tsx b/packages/react-shared/src/components/nested_sidebar_menu.tsx
deleted file mode 100644
index 196c91af1..000000000
--- a/packages/react-shared/src/components/nested_sidebar_menu.tsx
+++ /dev/null
@@ -1,119 +0,0 @@
-import { ObjectMap } from '@0x/types';
-import * as _ from 'lodash';
-import MenuItem from 'material-ui/MenuItem';
-import * as React from 'react';
-
-import { ALink, Styles } from '../types';
-import { colors } from '../utils/colors';
-import { utils } from '../utils/utils';
-
-import { Link } from './link';
-
-export interface NestedSidebarMenuProps {
- sectionNameToLinks: ObjectMap<ALink[]>;
- sidebarHeader?: React.ReactNode;
- shouldReformatMenuItemNames?: boolean;
-}
-
-export interface NestedSidebarMenuState {
- scrolledToId?: string;
-}
-
-const styles: Styles = {
- menuItem: {
- minHeight: 0,
- paddingLeft: 8,
- borderRadius: 6,
- },
- menuItemInnerDiv: {
- color: colors.grey800,
- fontSize: 14,
- lineHeight: 2,
- padding: 0,
- whiteSpace: 'nowrap',
- overflow: 'hidden',
- textOverflow: 'ellipsis',
- },
-};
-
-export class NestedSidebarMenu extends React.Component<NestedSidebarMenuProps, NestedSidebarMenuState> {
- public static defaultProps: Partial<NestedSidebarMenuProps> = {
- shouldReformatMenuItemNames: true,
- };
- private _urlIntervalCheckId: number | undefined = undefined;
- constructor(props: NestedSidebarMenuProps) {
- super(props);
- this.state = {};
- }
- public componentDidMount(): void {
- this._urlIntervalCheckId = window.setInterval(() => {
- const scrollId = location.hash.slice(1);
- if (scrollId !== this.state.scrolledToId) {
- this.setState({
- scrolledToId: scrollId,
- });
- }
- }, 200);
- }
- public componentWillUnmount(): void {
- window.clearInterval(this._urlIntervalCheckId);
- }
- public render(): React.ReactNode {
- const navigation = _.map(this.props.sectionNameToLinks, (links: ALink[], sectionName: string) => {
- const finalSectionName = utils.convertCamelCaseToSpaces(sectionName);
- // tslint:disable-next-line:no-unused-variable
- return (
- <div key={`section-${sectionName}`} className="py1" style={{ color: colors.greyTheme }}>
- <div style={{ fontSize: 14, letterSpacing: 0.5 }} className="py1 pl1">
- {finalSectionName.toUpperCase()}
- </div>
- {this._renderMenuItems(links)}
- </div>
- );
- });
- return (
- <div>
- {this.props.sidebarHeader}
- <div>{navigation}</div>
- </div>
- );
- }
- private _renderMenuItems(links: ALink[]): React.ReactNode[] {
- const scrolledToId = this.state.scrolledToId;
- const menuItems = _.map(links, link => {
- const finalMenuItemName = this.props.shouldReformatMenuItemNames
- ? utils.convertDashesToSpaces(link.title)
- : link.title;
- let menuItemStyle = styles.menuItem;
- let menuItemInnerDivStyle = styles.menuItemInnerDiv;
- const isScrolledTo = link.to === scrolledToId;
- if (isScrolledTo) {
- menuItemStyle = {
- ...menuItemStyle,
- backgroundColor: colors.lightLinkBlue,
- };
- menuItemInnerDivStyle = {
- ...menuItemInnerDivStyle,
- color: colors.white,
- fontWeight: 'bold',
- };
- }
- return (
- <div key={`menuItem-${finalMenuItemName}`}>
- <Link to={link.to} shouldOpenInNewTab={link.shouldOpenInNewTab}>
- <MenuItem style={menuItemStyle} innerDivStyle={menuItemInnerDivStyle}>
- <span
- style={{
- textTransform: this.props.shouldReformatMenuItemNames ? 'capitalize' : 'none',
- }}
- >
- {finalMenuItemName}
- </span>
- </MenuItem>
- </Link>
- </div>
- );
- });
- return menuItems;
- }
-}