Conversation
非实时渲染在 canvas 下面放置一个 本次 pr 欠缺之处：
|
| return self | ||
| }; | ||
|
|
||
| self.updateImage = function (isInit) { |
There was a problem hiding this comment.
能否有办法避免在库中setData呢，因为这样处理的话，和外部cropperOpt耦合了，能否做成外部的拓展模块，因为imgLeft、imgTop等属性是可以通过实例拿到的
There was a problem hiding this comment.
如果要在外部 page 中更新 UI 的话，这就可以解决 cropperOpt 耦合的问题了。可以在 __oneTouchMove 和 __twoTouchMove 中放置两个钩子函数，比如 onTouchMove， 然后开发者在自己的 page 中调用钩子函数来 setData。
这样会有个问题： <image/> 要由开发者去处理。以及 <image/> 是放在 template 里，还是开发者自己写在 page 里。
更好的方式应该是：开发者在使用 we-cropper 时，只需指定参数和监听事件，其他裁切相关的具体逻辑都无需过问。
如果将 cropperOpt 定义为固定的接口参数，那在 we-cropper 里 setData应该就不是问题。只需在裁切成功、失败以及取消之后，再 we-cropper.pageContext = null 来解决循环引用导致的内存泄漏问题。
如果以小程序的 component 组件 来实现 we-cropper，是不是会更合适
There was a problem hiding this comment.
这里我的考虑是，we-cropper理念还是设计成一个库，最终可以运行在各个平台（框架）上，尽可能脱离宿主上下文环境，目前we-cropper缺乏一套可靠的插件拓展机制，这个是一个必须解决的首要问题。
There was a problem hiding this comment.
更新你的代码以后，跑不起来呀，cropperImageSrc imageTop 这些属性都是获不到值，例子，应该是1.2版本的吧，
|
非实时渲染是否可以考虑对selft.updateCanvas() 做节流处理,这样子是否就可以不依赖于外部page 的image 标签? 做个trace 对比如何？ |
|
[小心翼翼地]问下这个准备发新版本了吗？ |
|
文件特别大，选择原图，移动和裁切都非常卡 |
5 participants
No description provided.