Closed xiaoyouPrince closed 1 month ago
longitachi
commented
2 months ago 感谢pr。
这个clipRatios的数组，本意就是裁剪图片时候支持的所有比例，就是要把用户设置的全部带入，跟预览图片几张没有关系。
并不是 选了三张图片，设置了三个比例，然后根据预览的index去取比例去裁剪。 两个数据没有对应关系， 而且可以选择10张甚至更多图片，但裁减比例可能就只有很少的几个。
我理解大多数情况下编辑图片都是统一的比例需求，应该很少有第一张图片1:1，第二张1:2，第三张3:4之类的需求吧
xiaoyouPrince
commented
2 months ago 哈哈, 感谢回复哈 我用到的这个功能确实是真实需要. 场景如下: (自定义壁纸: 背景 + 自定义图片内容)
我们在编辑页面可以批量上传多张图, 且每张图有自己各自的模版宽高比. 我最初设置的 clipRatios 也是与可选最大张数对应的. 从预览进裁剪页面需要用对应模版指定的 ratio 这样才能不走形.
我能理解你的解释和咱们代码的默认实现, 但是我遇到的问题也是真实场景, 我的 commit 也是在不改变默认实现的前提下提供了这个能力. 希望作者也能考虑一下.
如果你看代码本身符合规范, 还请考虑一下通过合并, 如果我的代码有没有考虑到的问题, 麻烦你考虑增加一下这个能力 (目前我 fork 了你仓库在使用, 之后还是希望直接使用本仓库.)
我目前的工作需要大量用到本项目, 可能还会有此类真实场景下的需求, 后续可能继续提交 pr, 如果能为咱这个优秀的开源项目贡献一点代码, 我自己也会非常荣幸 🤝
xiaoyouPrince
commented
2 months ago 作者你好,
我当前需求中有个新需求, 即弹出 ZLPhotoPreviewSheet 时候请求相册权限, 用户手动拒绝之后, 不再弹出去设置 alert, 我看到咱们项目中目前的操作是监听到用户的拒绝操作, 给了一个系统弹框, 并回调了 noAuthorityCallback.
基于我的需求和代码当前的实现, 我新增了 customAlertWhenNoAuthority: (() -> Void) 配置变量, 来提供一个让用户自定义无权限 alert 的能力.
希望作者看到后, 能考虑一下这个能力, 考虑一下通过 pr, 🤝
使用中发现， 批量选图之后，预览，编辑某张图， 会将所有 clipRatio 带到编辑页面， 实际我只想使用指定 ratio 编辑具体图。
此场景我在图片编辑的配置中，新增了 showCurrentRatioWhenBatchClip 属性。
如果代码符合作者要求，请合并一下， 🙏