新版 seajs-debug 的一些建议
afc163 opened this issue · 11 comments
想到先写下来备忘。
- 代码用等宽字体。
- 扁平化样式风格。
- 字体和按钮稍稍放大点。
- 输入框 placeholder。
- 这里应该是硬编码:https://github.com/seajs/seajs-debug/blob/master/src/seajs-debug.js#L533
- 下面的按钮作为常用功能,是否应该放在面板**的醒目位置,而 map 功能相对次要。
- 显示模块数和 seajs 加载性能。
- 显示 seajs-health 的相关信息?
写到 7 8 点突然想到,seajs 调试插件是不是就应该把 seajs 运行时的一些数据(模块数、加载性能、依赖深度、是否循环依赖)显示在面板的固定区域。所以 seajs-health 是不是应该和 seajs-debug 进行合并,这俩本来都是一个场景下的应该提供的功能。
合并的话感觉略大吧,debug 只做调试比较简单吧。
都是调试场景,感觉分两个插件没必要,不利于用户认知。
health 主要的还是状态展示,不只是为了排错,做的好可能需要很多代码。
显示 seajs-health 的相关信息?
觉得应该支持, 默认可以不显示, 但是可以通过一个按钮啥的, 可以加载进来呀!
同意 @leoner 的, 可以类似加载 seajs-log 那样, 点击一个按钮, 就把 seajs-health 加载进来, 但功能上 debug 和 health 还是分开好.
- 代码用等宽字体。
采用 font-family: Courier, monospace;
- 扁平化样式风格。
换了个皮肤, 紫色的, @afc163 您瞧瞧看?
- 字体和按钮稍稍放大点。
文本字体调成 14px , 标题 16px
- 输入框 placeholder。
已加, 不过之前也是有 title 的
这个是要给出 seajs-log 的 url, 在插件加载方式 #794 中列出的三种方式, 如果换成 url 上加 seajs-log 的话, 也是需要用户指定 alias 告诉 seajs seajs-log 插件在哪, 同 preload 给出 seajs-log 的 url 一样. 没想到其他的方式. 求更好的方式
- 下面的按钮作为常用功能,是否应该放在面板**的醒目位置,而 map 功能相对次要。
放中间太丑, 现在 icon 调大到 16px
panel 的样式是不是可以打包到 js 里?
输入框还挺好用的,当作简单的 console 了。
https://github.com/seajs/seajs-debug/blob/master/src/seajs-debug.js#L75
https://github.com/seajs/seajs-debug/blob/master/src/seajs-debug.js#L80
引其他插件的是不是不该硬编码?
就 preload,然后配 alias 不行?不配的话会 404,也很容易发现。
这个除了 health 的问题外, 都已修复.