Skip to content
This repository has been archived by the owner on Oct 20, 2021. It is now read-only.

refactor(site): make navbar scroll separately on webpage #7

Merged
merged 6 commits into from
Sep 3, 2018

Conversation

Kimi-Gao
Copy link
Member

@Kimi-Gao Kimi-Gao commented Aug 29, 2018

  • Make sure that you propose PR to right branch: bugfix for master, feature for latest active branch feature-x.x.
  • Make sure that you follow earth-ui's code convention.
  • Rebase before creating a PR to keep commit history clear.
  • Add some descriptions and refer relative issues for you PR.

Refactor :

  • make navbar scroll separately on webpage.
  • refactor website and rename lots of files.
  • remove some redundant code in site dir.
  • replace withRouter with react-router context.

Reference Issue :
#6

@Kimi-Gao Kimi-Gao added PR: refactor A code change that neither fixes a bug nor adds a feature tag: refactor labels Aug 29, 2018
@Kimi-Gao Kimi-Gao self-assigned this Aug 29, 2018
}
return (
<NavItem key={item.name} id={item.name}>
<span><span>{item.name}</span><span className="chinese">{item.cn}</span></span>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

嵌套了两层 span ,这里是必须的吗?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不小心多写了一个

Copy link
Collaborator

@weiyuxuan weiyuxuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Kimi-Gao Kimi-Gao merged commit 97efaa5 into feature-0.2 Sep 3, 2018
@Kimi-Gao Kimi-Gao deleted the navbar-scroll-alone branch September 3, 2018 02:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
PR: refactor A code change that neither fixes a bug nor adds a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants