Merge pull request #20 from surfstudio/fix/snapshot

Fix - Snapshot issues

.github/workflows/deploy_docc.yml

@@ -43,10 +43,10 @@ jobs:
             --output-path docs;
 
       - name: Upload artifact
-        uses: actions/upload-pages-artifact@v1
+        uses: actions/upload-pages-artifact@v3
         with:
           path: 'docs'
 
       - name: Deploy to GitHub Pages
         id: deployment
-        uses: actions/deploy-pages@v1
+        uses: actions/deploy-pages@v3

.github/workflows/main.yml

@@ -17,7 +17,7 @@ jobs:
 
     steps:
     - uses: actions/checkout@v2
-    - name: CocoaPods
+    - name: Init
       run: |
         gem install xcpretty -N
         make init

.ruby-version

@@ -0,0 +1 @@
+3.3.6

Gemfile

@@ -8,7 +8,7 @@ git_source(:github) do |repo_name|
 end if Gem::Version.new(Bundler::VERSION) < Gem::Version.new('2')
 
 gem "fastlane", "2.226.0"
-gem 'cocoapods'
+gem 'cocoapods', "1.16.2"
 gem 'xcpretty', "0.4.0"
 gem 'generamba', github: 'surfstudio/Generamba', tag: '1.7.1'
 

Makefile

@@ -8,7 +8,6 @@ init:
 	-bundle install
 	-bundle exec pod repo update
 	-bundle exec pod install
-	-bundle exec generamba template install
 
 build:
 	bundle exec fastlane build clean:true

Rambafile

@@ -27,6 +27,6 @@ podfile_path: Podfile
 
 ### Templates
 templates:
-- {name: surf_mvp_coordinatable_module, local: 'generamba/templates/surf_mvp_coordinatable_module'}
-- {name: surf_mvp_coordinatable_alert, local: 'generamba/templates/surf_mvp_coordinatable_alert'}
-- {name: surf_mvp_coordinator, local: 'generamba/templates/surf_mvp_coordinator'}
+- {name: surf_mvp_coordinatable_module}
+- {name: surf_mvp_coordinatable_alert}
+- {name: surf_mvp_coordinator}

SurfPlaybook/Playbook/Core/Containers/SUIViewContainer.swift

@@ -61,6 +61,8 @@ extension SUIViewContainer: PlaybookContainer {
             return self
         }
 
+        contentView.backgroundColor = .clear
+
         translatesAutoresizingMaskIntoConstraints = false
         contentView.translatesAutoresizingMaskIntoConstraints = false
 

SurfPlaybook/Playbook/Core/PlaybookPage.swift

@@ -40,15 +40,23 @@ public struct PlaybookPage {
         return config(nil).makeSnapshot()
     }
 
+    /// Предпочтительная конфигурация.
+    /// При наличии вариантов конфигурации предпочтение отдаётся первой конфигурации.
+    /// Это сделано для того, чтобы в частном случае можно было в `config` записать
+    /// эксклюзивную конфигурацию для snapshot, с дополнительным фоном или отключенными анимациями.
+    var preferredConfig: PlaybookContainerProvider {
+        presets.first?.config ?? config
+    }
+
     // MARK: - Public Properties
 
     /// Массив, содержащий различные варианты конфигурации UI-компонента.
     ///
     /// Представляет собой массив элементов, где каждый состоит из двух компонентов
-    /// - preset - абстрактный объект, выполняющий протокол `StringRepresentable`,
+    /// - **preset** - абстрактный объект, выполняющий протокол `StringRepresentable`,
     ///     где протокол содержит лишь одно поле `title`. Так что, фактически,
     ///     это просто `название состояния`
-    /// - config - замыкание, при вызове которого строится нужное состояние UI-компонента
+    /// - **config** - замыкание, при вызове которого строится нужное состояние UI-компонента
     ///     и возвращается UIView, его содержащее
     public var presets: [(preset: StringRepresentable, config: PlaybookContainerProvider)] = []
 
@@ -57,11 +65,15 @@ public struct PlaybookPage {
     /// Конструктор для создание страницы плейбука
     ///
     /// - Parameters:
-    ///     - name: название UI-компонента, будет отображено в приложении
-    ///     - description: опциональное описание компонента
+    ///    - name: название UI-компонента, будет отображено в приложении
+    ///    - description: опциональное описание компонента
     ///         (пока что не используется и нигде не отображается)
-    ///     - config: замыкание, при вызове которого должно строится нужное состояние UI-компонента
+    ///    - config: замыкание, при вызове которого должно строится нужное состояние UI-компонента
     ///         и возвращаться UIView, его содержащее
+    ///  - Warning: Снимок для превью компонента делается с `config`, а не с `presets`.
+    ///  В случае, если UI-компонент **анимирован** и анимации **искажают** его preview или оставляют его **пустым**,
+    /// можно передать в `config` уникальную конфигурацию с отключенной анимацией,
+    /// а конфигурацию для показа установить в `presets`.
     public init(name: String,
                 description: String?,
                 config: @escaping PlaybookContainerProvider) {

SurfPlaybook/Playbook/Flows/Main/Page/View/PageViewController.swift

@@ -59,7 +59,7 @@ private extension PageViewController {
 
     func configureAppearance(with page: PlaybookPage) {
         configureNavigationBar(with: page)
-        configureContent(contentView: page.config(self).loadView())
+        configureContent(contentView: page.preferredConfig(self).loadView())
         configureGesture()
     }
 

SurfPlaybook/Playbook/Flows/TabBar/TabBar/View/TabBarViewController.swift

@@ -67,6 +67,21 @@ extension TabBarViewController: UITabBarControllerDelegate {
             isStackEmpty = navigationController?.visibleViewController == navigationController?.viewControllers[0]
         }
         isSelectableStackEmpty = isStackEmpty
+
+        // Отключаем новомодную [анимацию](https://medium.com/@adityaramadhan.biz/new-tabbar-transition-animation-in-ios-18-and-xcode-16-ea4b2c4d84d4)
+        guard let fromView = selectedViewController?.view,
+              let toView = viewController.view else {
+            return false
+        }
+
+        if fromView != toView {
+            UIView.transition(from: fromView,
+                              to: toView,
+                              duration: 0,
+                              options: [.transitionCrossDissolve],
+                              completion: nil)
+        }
+
         return true
     }