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

优化点整理 #243

Open
11 tasks
fengzilong opened this issue Jan 16, 2019 · 18 comments
Open
11 tasks

优化点整理 #243

fengzilong opened this issue Jan 16, 2019 · 18 comments
Assignees

Comments

@fengzilong
Copy link
Member

fengzilong commented Jan 16, 2019

  • watcher 的调试

    无法看到真实的表达式,断点打到这里,调试起来不是特别方便

  • 无法通过组件实例很快地联想到对应的组件

    group 这里可以拿到 element;template 因为被解析成 ast 了,所以无法通过 template 很直观地联想到是哪一个组件,虽然 regular-devtools 已经实现,但是不一定所有人都会安装

  • ts友好度

  • hot reload

  • 模板报错提示优化

  • vscode等生态完善 ( autocomplete... )

  • 给出 circular dependency 细节,方便排查定位问题

  • 维护 CHANGELOG

  • 官方 awesome-regular

  • ESLint

  • 变量名优化,增加注释,方便维护

持续整理中...

看到这个 issue 的同学,觉得哪方面不足的也可以在下面回复,会优先解决

@leeluolee
Copy link
Member

补充下,同事反馈比较多的.

  • ts 友好度
  • hot reload

@XGHeaven
Copy link

PS: 我这里是老版本,0.5.2,如果新版本修复了,就忽略好了。

  • 模板报错提示,无法很轻松的定位到到底是哪个模板出错了。
  • 模板提示,虽然有 vscode 插件,但是无法很好的提示究竟有哪些变量,哪些方法。而且我们会把模板拆出来一个 html 文件,就更难提示了。

@Qquanwei
Copy link

感觉计算属性的报错不易定位. 计算属性报错为检测到 circular dependency.

@fengzilong
Copy link
Member Author

模板报错定位这个,我可以在 regular-loader 里面加下 file 路径

@fengzilong
Copy link
Member Author

fengzilong commented Jan 17, 2019

circular dependency 的原因描述的确可以再详细点

@XGHeaven
Copy link

@fengzilong 额,我们还在用 NEJ,比较恶心。另外只需要能够提示一下是哪个组件的模板的第几行报错了,或者给出报错的表达式也可以,这个其实不怎么需要 loader 的参与。

@fengzilong
Copy link
Member Author

fengzilong commented Jan 17, 2019

@XGHeaven 想要知道哪个组件必须要借助构建工具,不过模板的第几行报错应该是可以拿到的

#200

0.5.x 的 regular 报错位置有点问题,0.6.0 以后的版本应该修复这个问题了

@fengzilong
Copy link
Member Author

优化点补充:

维护一份 CHANGELOG

@Qquanwei
Copy link

Qquanwei commented Jan 17, 2019

circular dependency 参考 #225

@XGHeaven
Copy link

@fengzilong 为什么一定要借助构建工具才能拿到组件呢?可以在编译模板的时候,传入组件信息,得到的渲染函数就可以自己捕获错误显示组件信息。或者在渲染模板的也可以~ :)

@Qquanwei
Copy link

请问目前有 awesome-regularjs 系列吗。没有的话可以维护一份。

@fengzilong
Copy link
Member Author

fengzilong commented Jan 17, 2019

嗯嗯,circular dependency 这个我有遇到过的

https://github.com/fengzilong/awesome-regular
@Qquanwei 我自己很早前有整理过一份的,不过还不全,后面考虑转到这里,方便大家一起补充

@fengzilong
Copy link
Member Author

fengzilong commented Jan 17, 2019

@XGHeaven 可能我和你说的不是同一个错误捕获,我指的是编译模板时候的错误,你指的是模板编译后发生的错误么?

@fengzilong
Copy link
Member Author

fengzilong commented Jan 17, 2019

优化点补充:

  • jshint -> ESLint,找个时间把代码格式化下
  • 部分目录结构调整,变量命名调整,增加更多的注释

@fengzilong
Copy link
Member Author

@XGHeaven 模板编译的报错可能做个编辑器插件更好,可以实时看到报错

@leeluolee
Copy link
Member

@fengzilong 这个awesome系列确实不太awesome... 其实确实有些零散资源,可以整合个入口入口看看。 比如 https://github.com/chenshy39/regular-autocomplete 这种编辑器插件

@fengzilong
Copy link
Member Author

@leeluolee 嗯嗯,之前就开了个头,的确不太 awesome,后面再收集看看补充上去

@XGHeaven
Copy link

@fengzilong 哦哦,我懂你意思了。主要我们现在用 NEJ,所以只需要有正确的运行时的模板报错提示就足够了。不过要是等以后可能重构了,准没就用 react。

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

No branches or pull requests

4 participants