search

surfstudio / ReactiveDataDisplayManager

MIT License
34 stars 13 forks source link

SPT-1455 Сделать Foldable plugin доступным для BaseManager #214

Closed Golubeykov closed 1 year ago

Golubeykov commented 1 year ago

SPT-1455

Что сделано?

Сделал Foldable plugin доступным для BaseManager

Зачем это сделано?

Ранее данный плагин был доступен только для ManualManager, но он не удобный. Большинство пользователей выбирает BaseManager.

Как протестировать?

1) Установить патч ReactiveDataDisplayManager-18-23-40.patch 2) Таб Table 3) Table with foldable cell

Демо

https://github.com/surfstudio/ReactiveDataDisplayManager/assets/47087482/caad402a-008c-4f4b-a45e-1ea0ef937f1f

github-actions[bot] commented 1 year ago
Warnings
:warning: Oops! We have found some issues. It's better to fix them to keep code clean

SwiftLint found issues

Severity File Reason
Warning FoldableItem.swift:25 Lines should not have trailing whitespace. (trailing_whitespace)
Warning BaseTableManager.swift:190 Lines should not have trailing whitespace. (trailing_whitespace)

Generated by :no_entry_sign: Danger Swift against 34a58ca947c1e35b62d52c1a1d235ac91cedb9a4

Ikeret commented 1 year ago

Патч для того, что на ветку не переключаться?)

Golubeykov commented 1 year ago

Патч для того, что на ветку не переключаться?)

Не, я заменил manualBuilder на base в приложении-примере. Не знаю, можно ли так было делать, поэтому вынес в патч)