提几个优化建议

[steveleeh]

很感谢作者的分享出来这个组件，提几个小的建议：

1. SliderProps 里面的widtih现在都是number，并且依赖于这个width做计算，但是如果希望这个width是100%，撑满外层盒子，这个设计就有问题了，因为width必须强制指定，还要是number。
2. 很多字段设计的多余，borderColor， borderRadius，这个完全可以把盒子的className和style暴露出来，让别人传进去自己决定，没必要每个都指定，也指定不全。
3. 滑块如果能暴露出来一个renderSlider（React.ReactNode）类似的props，能覆盖默认的渲染内容就好了，相应的文案部分也可以这样暴露，自定义就比较高了。

[guqingming]

font{
    line-height: 1.6;
}
ul,ol{
    padding-left: 20px;
    list-style-position: inside;
}

感谢提的相关建议，因为这个组件的初衷是简易，快速实现一个滑动验证效果，所以都是怎么简单怎么来的，而且这个组件是vue前身，只是同功能迁移到了react的，所以很多参数的设定都是按照vue组件的开发习惯来的

                            15524261

                                15524261@qq.com

    签名由
    网易邮箱大师
    定制

在2020年10月23日 10:31，steveleeh<notifications@github.com> 写道： 

很感谢作者的分享出来这个组件，提几个小的建议：

SliderProps 里面的widtih现在都是number，并且依赖于这个width做计算，但是如果希望这个width是100%，撑满外层盒子，这个设计就有问题了，因为width必须强制指定，还要是number。 很多字段设计的多余，borderColor， borderRadius，这个完全可以把盒子的className和style暴露出来，让别人传进去自己决定，没必要每个都指定，也指定不全。 滑块如果能暴露出来一个renderSlider（React.ReactNode）类似的props，能覆盖默认的渲染内容就好了，相应的文案部分也可以这样暴露，自定义就比较高了。

—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.