Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

brix.js 是否有必要持有 ready 事件? #55

Open
nuysoft opened this issue Mar 18, 2013 · 1 comment
Open

brix.js 是否有必要持有 ready 事件? #55

nuysoft opened this issue Mar 18, 2013 · 1 comment

Comments

@nuysoft
Copy link
Member

nuysoft commented Mar 18, 2013

brix.js 是 Brix 应用的入口,主要提供了 3 项功能:

  1. 读取配置选项,与默认选项值合并,最终构造出完整的选项集。
  2. 提供 Brix.ready(fn) 功能,用以取代 KISSY.ready(),统一编码风格(推测)。
  3. 在选项 autoConfig 为 true 时配置包路径和包路径映射(map)。

在查看上述实现时,有 2 个疑问:

  1. brix.js 中涉及的初始化选项繁多,而且不集中,这个问题已向波哥反映,属于代码待重构改进。
  2. brix.js 是否有必要持有 ready 事件?个人没有觉得这么做带来了新功能或者方便,封装一次就多一个疑问多一点学习成本,因此建议仍然交给 KISSY.ready()。
@keyapril
Copy link
Member

KISSY ready 是在dom ready 时候调用,而Brix ready 是在pagelet 构建完成调用 ,两个的职责不一样。

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

No branches or pull requests

2 participants