search

antvis / S2

⚡️ A practical visualization library for tabular analysis.
https://s2.antv.antgroup.com
MIT License
1.46k stars 193 forks source link

PivotSheet滚动条hover时会变粗, 但显示不全 #2825

Open xuefeibai opened 1 month ago

xuefeibai commented 1 month ago

🏷 Version

Package Version
@antv/s2 1.49.1
@antv/s2-react
@antv/s2-vue 2.6.14

Sheet Type

🖋 Description

PivotSheet和TableSheet的滚动条, 在设置hoverSize为10时, 如果hover横向滚动条, 下半部分显示不全, 如果hover纵向滚动条, 右半部分显示不全 image image

⌨️ Code Snapshots

🔗 Reproduce Link

https://codesandbox.io/p/sandbox/s2-simple-yg9m4v 上面是一个示例沙盒, 代码使用的是lastest版本, 实际上我的生产环境使用的是1.49.1版本.

🤔 Steps to Reproduce

😊 Expected Behavior

hover滚动条, 滚动条自动变粗, 能显示完整

😅 Current Behavior

hover滚动条, 滚动条自动变粗, 但显示不全

💻 System information

Environment Info
System
Browser
github-actions[bot] commented 1 month ago

你好 @xuefeibai，感谢你的建议, 我们会及时评估和排期, 谢谢! 当然, 如果能贡献 PR 帮助我们改进, 不胜感激！

Hello, @xuefeibai, thanks for your advice. We will evaluate and schedule in time. Thank you! you could implement it by yourself through the customization capabilities provided by S2. Thanks so much for your understanding.

lijinke666 commented 1 month ago

相关问题: https://github.com/antvis/S2/issues/2295

lijinke666 commented 1 month ago

感谢反馈, 目前在 1.x 版本中只能建议暂时减小 size 和 hoverSize 的差异.

由于涉及到布局改造, 会带来 break change, 滚动条相关问题计划后续在 2.0 版本中重构解决:

  1. 滚动条位置不统一: 水平滚动条滑道在表格外, 而垂直滚动条在表格内
  2. size 和 hoverSize 差异过大时展示不全, 默认是按照初始大小计算坐标, 未考虑 hover 后.
  3. 无滚动条时不应该占位 https://github.com/antvis/S2/issues/2295
  4. 滚动条悬浮展示 https://github.com/antvis/S2/issues/2834
xuefeibai commented 1 month ago

我本地尝试修复, 在mouseover时, 重新计算scrollBarGroup的位置, 然后调用move移动位置, 结果滚动条不见了. 然后我换了一种方案, 初始化时新增一个hover态滚动条, 默认不显示, mouseover滚动条时进行切换, 从而解决了当前问题. 这种方案只能临时用用.

xuefeibai commented 1 month ago

嘶, 第一个方法是可以的, move的单位是偏移量, 而不是绝对位置~