Animated onboarding #201
Animated onboarding #201
Conversation
Ostrenkiy
left a comment
Ostrenkiy
left a comment
There was a problem hiding this comment.
Вообще хорошо, можно мерджить.
Но лучше поправить пару замечаний
| private var scrollView: UIScrollView! | ||
| fileprivate var pages: [OnboardingPageView] = [] | ||
|
|
||
| private var backgroundGradient: CAGradientLayer = { |
There was a problem hiding this comment.
Может это стоит вынести в какое-нибудь отдельное место? Кажется, это может много где пригодиться
There was a problem hiding this comment.
Вынести код создания градиента с заданным углом? Можно сделать какой-нибудь extension (например, для CAGradient).
There was a problem hiding this comment.
Да. Можно даже с одним углом (чтобы во всем приложении все градиенты были одинаковые).
В каталоге ведь такие же градиенты используются сейчас, кажется
| private var titles = (1...4).map { NSLocalizedString("OnboardingTitle\($0)", comment: "")} | ||
| private var descriptions = (1...4).map { NSLocalizedString("OnboardingDescription\($0)", comment: "")} | ||
|
|
||
| fileprivate var isLandscape: Bool { |
There was a problem hiding this comment.
Может это в DeviceInfo вынести?
There was a problem hiding this comment.
Вообще, тут немного в нейминге дело – он немного путает. Изначально планировалось, что это поле true для случаев, когда необходимо показывать лэйаут "как в лендскейпе". То есть включая лэндскейпы, но не ограничиваясь ими (в какой-то момент правда отпала необходимость в этом).
Задача: #APPS-1651
Коротко для Release Notes, в формате «Сделали/Добавили/Исправили N»:
Добавили анимированный онбординг
Описание:
Анимированный с помощью Lottie онбординг.
TODO