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

[Table] 支持第三方滚动条插件 #2207

Open
xmsz-stu opened this issue Apr 26, 2024 · 8 comments
Open

[Table] 支持第三方滚动条插件 #2207

xmsz-stu opened this issue Apr 26, 2024 · 8 comments
Labels
feature request Request new feature

Comments

@xmsz-stu
Copy link

Which Component Need Enhancement 期望新增功能的组件

  • Table

What does the proposed API look like 期望支持的API

支持滚动条插件,比如simplebar

Feature Description 功能描述

因为默认的滚动条在不同系统和浏览器下显示不一样,没办法实现样式统一
最恶心的是有些时候会占据宽度,有些时候又不会,占据宽度的情况下长这样
image
这一块直接是没颜色的

Additional information 补充说明

  • 希望涉及滚动条的都能支持第三方滚动插件

相关问题:#884

@xmsz-stu xmsz-stu added the feature request Request new feature label Apr 26, 2024
@shijiatongxue
Copy link
Collaborator

shijiatongxue commented May 16, 2024

你好,是希望我们默认加上这种 data 标识到所有的滚动轴相关的组件吗?

比如给 Table body 对应的 div 加上这个 data-simplebar 标识。

image

这个 simplebar-react npm 包下载量,每周 50w+,还是比较多的。

image

simplebar 每周下载量也不少。

image

@xmsz-stu
Copy link
Author

你好,是希望我们默认加上这种 data 标识到所有的滚动轴相关的组件吗?

比如给 Table body 对应的 div 加上这个 data-simplebar 标识。

image 这个 simplebar-react npm 包下载量,每周 50w+,还是比较多的。 image simplebar 每周下载量也不少。 image

就是可以用第三方插件替换原生的滚动条样式

@shijiatongxue
Copy link
Collaborator

你好,是希望我们默认加上这种 data 标识到所有的滚动轴相关的组件吗?
比如给 Table body 对应的 div 加上这个 data-simplebar 标识。
image
这个 simplebar-react npm 包下载量,每周 50w+,还是比较多的。
image
simplebar 每周下载量也不少。
image

就是可以用第三方插件替换原生的滚动条样式

我明白你想做什么,主要是怎么做。

@shijiatongxue
Copy link
Collaborator

你好,是希望我们默认加上这种 data 标识到所有的滚动轴相关的组件吗?
比如给 Table body 对应的 div 加上这个 data-simplebar 标识。
image
这个 simplebar-react npm 包下载量,每周 50w+,还是比较多的。
image
simplebar 每周下载量也不少。
image

就是可以用第三方插件替换原生的滚动条样式

比如需要 Semi 这边做什么能支持你做的事情。我们是不会内置这个第三方插件的(体积大小),这个不是一个基础组件库需要内置的包。

@xmsz-stu
Copy link
Author

你好,是希望我们默认加上这种 data 标识到所有的滚动轴相关的组件吗?
比如给 Table body 对应的 div 加上这个 data-simplebar 标识。
image
这个 simplebar-react npm 包下载量,每周 50w+,还是比较多的。
image
simplebar 每周下载量也不少。
image

就是可以用第三方插件替换原生的滚动条样式

比如需要 Semi 这边做什么能支持你做的事情。我们是不会内置这个第三方插件的(体积大小),这个不是一个基础组件库需要内置的包。

ok, 那可以试试data-simplebar能不能实现,我只找到两个demo
https://jsfiddle.net/1z76Lhvo/12/
https://codepen.io/sdthornton/pen/mdOaoML
但都是附在table的父级,感觉没啥用,不知道table的body能不能兼容

@shijiatongxue
Copy link
Collaborator

你好,是希望我们默认加上这种 data 标识到所有的滚动轴相关的组件吗?
比如给 Table body 对应的 div 加上这个 data-simplebar 标识。
image
这个 simplebar-react npm 包下载量,每周 50w+,还是比较多的。
image
simplebar 每周下载量也不少。
image

就是可以用第三方插件替换原生的滚动条样式

比如需要 Semi 这边做什么能支持你做的事情。我们是不会内置这个第三方插件的(体积大小),这个不是一个基础组件库需要内置的包。

ok, 那可以试试data-simplebar能不能实现,我只找到两个demo https://jsfiddle.net/1z76Lhvo/12/ https://codepen.io/sdthornton/pen/mdOaoML 但都是附在table的父级,感觉没啥用,不知道table的body能不能兼容

可以,近期有其他高优的事情,这个需要晚点看下。

@xmsz-stu
Copy link
Author

我刚刚试了用

 new SimpleBar(document.query.("... .semi-table-body"));

来初始化
样式是成功替换上了

但是滚动不会加载了 相当于虚拟滚动失效了

@shijiatongxue
Copy link
Collaborator

我刚刚试了用

 new SimpleBar(document.query.("... .semi-table-body"));

来初始化 样式是成功替换上了

但是滚动不会加载了 相当于虚拟滚动失效了

可以试下,在 semi 仓库拉到本地,用 data-simplebar 的方式看看是否生效。如果OK的话,欢迎提个 pr。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Request new feature
Projects
None yet
Development

No branches or pull requests

2 participants