search

jasl / cybros_core

A demo app to show some Rails app configurations
MIT License
122 stars 26 forks source link

使用stimulus重构coreui的sidebar #11

Closed monsterooo closed 5 years ago

monsterooo commented 5 years ago

Fix: #10 #7

针对coreui使用stimulus进行重构

修改如下：

monsterooo commented 5 years ago

还有比如setActiveLink 一些功能晚上再进行重构 😀

jasl commented 5 years ago

你觉得把UI框架上的功能放到一个 FrameworkController 如何？

monsterooo commented 5 years ago

你觉得把UI框架上的功能放到一个 FrameworkController 如何？

嗯, 可以的。 但是这种独立的组件建议单独放到一个controller中, 以后重构或者复用比较友好

jasl commented 5 years ago

你觉得把UI框架上的功能放到一个 FrameworkController 如何？

嗯, 可以的。 但是这种独立的组件建议单独一个控制, 以后重构或者复用比较友好

明白了

monsterooo commented 5 years ago

coreui给sidebar滚动条增加了一个自定义的滚动条样式，是否去掉这个功能？ 因为我觉得这个用处不大，仅仅是给滚动条增加了一个自定义样式

jasl commented 5 years ago

coreui给sidebar滚动条增加了一个自定义的滚动条样式，是否去掉这个库？ 因为我觉得这个用处不大，仅仅是给滚动条增加了一个自定义样式

我都没注意到... 去掉吧，CoreUI 花哨的东西挺多的，sidebar 收缩的动效在我的 i9 MBP 上都卡...

jasl commented 5 years ago

对了，还有个功能要加个，就是要用 Cookie 或者 LocalStorage 之类的东西记录一下sidebar的状态，这样刷新页面的时候能够保持用户上一次设置的样子

我本来想引用一下 Gitlab 的例子，但我刚才翻他的代码，我发现我已经看不懂了... https://gitlab.com/gitlab-org/gitlab-ce/tree/master/app/assets/javascripts/sidebar 一个 Sidebar 搞出十几个文件来

monsterooo commented 5 years ago

@jasl sidebar展开和收缩的状态都要进行状态保留对吧？ 如果是这样可以记录在LocalStorage里头没问题

jasl commented 5 years ago

@jasl sidebar展开和收缩的状态都要进行状态保留对吧？ 如果是这样可以记录在LocalStorage里头没问题

是的