iTofu / LCActionSheet

一款简约而不失强大的 ActionSheet,微博、微信和 QQ 都采用了极其类似的样式,完全支持 Swift。
https://note.leodev.me/LCActionSheet/
MIT License
821 stars 144 forks source link

建议所有配置都可以一次性设置而不是每个实例都配置一遍。 #29

Closed seekcx closed 7 years ago

seekcx commented 7 years ago

类似SVProgressHUD。

大多数项目为了达到视觉一致,都会统一进行设置的。

但是目前这样,每个实例设置一遍不仅冗余代码多,而且难以维护。

另:建议增加title contentInset设置,类似微信目前的title比较高,留白较多,看起来会更加大气。

iTofu commented 7 years ago

是的,谢谢你的反馈,这是一个可改进点。

关于统一配置,我之前考虑过这个问题,想法如下,你可以帮我一块看看是否满足需求:

增加一个 Config 类,它提供一个单例,用来统一配置参数。LCActionSheet 提供一个 -initWithConfig: 方法,在初始化时用该方法取代 -init: 方法,并传入 Config 单例,这样在初始化时可以直接配置好 Config 中的参数。初始化之后,如果开发者需要,也可以继续对 LCActionSheet 实例修改属性。简单代码如下:

至于 title contentInset 这个需求我记下来,时间充足的话尽快实现,最近公司业务有点忙。

seekcx commented 7 years ago

恩,这种方式挺好的。

还有一个问题: 貌似swift里不能通过实例调用appendButtonTitles这个方法。

iTofu commented 7 years ago

嗯,回头我看一下。

发自我的 iPhone

在 2016年11月28日,下午6:59,abel notifications@github.com 写道:

恩,这种方式挺好的。

还有一个问题: 貌似swift里不能通过实例调用appendButtonTitles这个方法。

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

iTofu commented 7 years ago

Hi,全局配置和 titleEdgeInsets 都已实现,更新到 V 2.7.0 就可以了!我测试 ok,你试一下看有没有问题。

iTofu commented 7 years ago

-appendButtonTitles: 方法貌似是 Swift 没有桥接成功……回头再看吧……

seekcx commented 7 years ago

恩恩,目前这样就只需要配置一次就可以了,而且titleEdgeInsets实现了,非常棒👍