Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

旋转屏幕的时候, 遮罩视图只遮住了一般。 #3

Closed
laughingg opened this issue Dec 15, 2016 · 1 comment
Closed

旋转屏幕的时候, 遮罩视图只遮住了一般。 #3

laughingg opened this issue Dec 15, 2016 · 1 comment

Comments

@laughingg
Copy link

laughingg commented Dec 15, 2016

snip20161215_23

  1. 使用了三个 tableview . 可以考虑一下是否方案太臃肿。 可以考虑 UIStackView
  2. 三个弹出视图可以抽取一个父类出来。 这样可以省去很多代码
  3. HY_Constants 没有必要继承自 UIView . HY_Constants 没有用到 UIView 的任何东西。 单纯是一个工具类,不考虑继承可以使用结构体。
  4. 内部布局用到了 SnapKit lib . 从你布局代码的书写来看。 你可以自己以系统 autolayout 为基础写几个加约束的工具方法。(不是特别依赖其他第三方库,不建议在自己的第三方库中使用第三方库)
    只有 HYAlertCell 和 HYShareCollectionCell 中的少量布局代码使用了 SnapKit
@iafine
Copy link
Owner

iafine commented Dec 15, 2016

谢谢建议!最近太忙,准备年底出一个稳定的版本,可能会有大的改动,包括设计结构、语法上可能会更Swift化,你的建议也在我的计划之内^_^

@iafine iafine closed this as completed Dec 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants