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

add topBarVisible config #765

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

TomGarden
Copy link

@TomGarden TomGarden commented May 22, 2019

提交这个 pr 的原因:

  • 在是用到滑动监听接口的时候认为顶部的按钮和 title 对我的需求而言是多余的,从而萌生了增加此配置功能的念头。
  • 即使我的实现有问题这个配置需求的变更也是有意义的。希望能考虑下。

我对代码做的修改:

  1. 在 PickerOptions 文件中添加了标志位 topBarVisible(是否显示标题栏? 默认 VISIBLE)
  2. 其他文件的修改以此为中心展开
  3. 在 app(moudle) 中添加了一个此效果的示例

如果在 master 没问题的话

2. 其他文件的修改以此为中心展开
3. 在 app(moudle) 中添加了一个此效果的示例
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