让添加按钮后的 control bar 能够滚动 #4345
Open
让添加按钮后的 control bar 能够滚动 #4345
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
实测触摸无效, 而且跟其他一些控制栏相关功能 (控制栏背景, 触摸优化之类的) 有冲突, 我觉得这个得单独弄个组件比较好, 明确说明和哪些组件不兼容. (去兼容这些也相当头疼, 而且可能对你来说也派不上用场)
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
我的笔记本上使用的分辨率不太高，document.body.offsetWidth 只有 1275px。所以在往 control bar 添加按钮后显示会有问题：
于是我为 control bar 添加了滚动功能
鼠标滚动已经经过测试。但我手上没有平板，没办法测试触摸操作，希望能帮忙测试一下。
也可以考虑，把这个东西独立成一个组件。分辨率不大的用户自行安装；毕竟添加了几个监听事件，还是有些性能成本的。