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

使用stimulus重构coreui的sidebar #11

Merged
merged 2 commits into from
Jul 8, 2019

Conversation

monsterooo
Copy link

Fix: #10 #7

针对coreui使用stimulus进行重构

修改如下:

  • 取消对coreui的import

  • 增加一个sidebar控制器用来切换侧边栏

@jasl jasl self-assigned this Jul 8, 2019
@monsterooo
Copy link
Author

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

@jasl
Copy link
Owner

jasl commented Jul 8, 2019

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

@monsterooo
Copy link
Author

monsterooo commented Jul 8, 2019

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

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

@jasl
Copy link
Owner

jasl commented Jul 8, 2019

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

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

明白了

@monsterooo
Copy link
Author

monsterooo commented Jul 8, 2019

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

@jasl
Copy link
Owner

jasl commented Jul 8, 2019

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

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

@jasl
Copy link
Owner

jasl commented Jul 8, 2019

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

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

@monsterooo
Copy link
Author

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

@jasl
Copy link
Owner

jasl commented Jul 8, 2019

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

是的

@jasl jasl merged commit 8c9b7dd into jasl:stimulus Jul 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants