be-fe / iSlider

Smooth mobile touch slider for Mobile WebApp, HTML5 App, Hybrid App
http://be-fe.github.io/iSlider/
MIT License
1.66k stars 448 forks source link

亲,这句话是不是有点问题 #409

Closed geeeger closed 7 years ago

geeeger commented 7 years ago
    this.fire('initialize');
    this._renderWrapper();
    this._initPlugins();
    this._bindHandler();
    this.fire('initialized');

在iSlider的构造器里,发出去了初始化和初始化完成事件。但事实上我感觉这两个事件没什么卵用啊。因为这两个事件是在实例化过程中发出去的,我要调用on去注册事件监听这两个事件的话,就必须等实例化完。讲道理我是怎么也接不到这两个事件的。同理还有'pluginInitialize','pluginInitialized'。🙈

shinate commented 7 years ago

这么写是因为要统一事件的注册和调用,任何事件都是用on来注册,fire调用。 initialize、initialized、pluginInitialize、pluginInitialized 由于触发位置的关系,要在构造时的options里面设置。

shinate commented 7 years ago

https://github.com/be-fe/iSlider#event-callbacks

geeeger commented 7 years ago

😂啊,768行的events初始化过程看掉了,不好意思。