Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AnchorSmooth,GoTop]:添加两个锚点相关的功能,以及中英文版的markdown文档使用说明。 #945

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Next-One
Copy link

@Next-One Next-One commented Mar 1, 2019

1.AnchorSmooth:实现点击AnchorSmooth滑动到相应的锚点,而不是直接跳转;
2.GoTop:可以定制的回顶部功能按钮,包括按钮的出现时间,回到顶部时执行的回调函数。
*问题:GoTop组件需要的一些样式,直接放在了anchor目录下,没有做成默认样式,供用户选择。为了通过test将GoTop中导入的样式删除,在markdown文件中没有效果。
*说明:看过pr指南,并全部通过,才提交pr的。

Please makes sure these boxes are checked before submitting your PR, thank you!

  • Make sure you are merging your commits to master branch.
  • Add some descriptions and refer relative issues for you PR.
  • Rebase your commits to make your pull request meaningful.
  • Make sure that your changes pass npm test, npm run lint and npm run build.

Changes in this pull request

  • 添加两个锚点相关的功能:1.AnchorSmooth:实现点击AnchorSmooth滑动到相应的锚点,而不是直接跳转;2.GoTop:可以定制的回顶部功能按钮,包括按钮的出现时间,回到顶部时执行的回调函数。
  • 中英文版的markdown文档使用说明,包括:en-US/anchor-smooth.md文件;en-US/go-top.md文件;zh-CN/anchor-smooth.md文件;zh-CN/go-top.md文件;
  • 文件命名合理,并通过 npm test, npm run lint and npm run build命令测试。
  • 对已有功能未做任何修改。

1.AnchorSmooth:实现点击AnchorSmooth过渡到相应的锚点;
2.GoTop:可以定制的回顶部功能按钮,包括按钮的出现时间,回到顶部时执行的回调函数。
问题:GoTop组件需要的一些样式,直接放在了anchor目录下,没有做成默认样式,供用户选择。
1.AnchorSmooth:实现点击AnchorSmooth过渡到相应的锚点;
2.GoTop:可以定制的回顶部功能按钮,包括按钮的出现时间,回到顶部时执行的回调函数。
问题:GoTop组件需要的一些样式,直接放在了anchor目录下,没有做成默认样式,供用户选择。
1.AnchorSmooth:实现点击AnchorSmooth滑动到相应的锚点,而不是直接跳转;
2.GoTop:可以定制的回顶部功能按钮,包括按钮的出现时间,回到顶部时执行的回调函数。
问题:GoTop组件需要的一些样式,直接放在了anchor目录下,没有做成默认样式,供用户选择。
@Next-One Next-One changed the title [AnchorSmooth,GoTop]:添加两个锚点相关的功能,以及中英文版的使用说明markdown文档。功能如下: [AnchorSmooth,GoTop]:添加两个锚点相关的功能,以及中英文版的markdown文档使用说明。 Mar 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant