felixfoertsch/bundesmessenger-ios
1
0
Fork 0
mirror of https://gitlab.opencode.de/bwi/bundesmessenger/clients/bundesmessenger-ios.git synced 2026-04-22 17:42:45 +02:00
Code Issues Packages Projects Releases Wiki Activity

[Spaces] M10.6 Space preview bottom sheet #4497

Browse Source 
- Update after code review
This commit is contained in:
Gil Eluard
2021-09-15 11:16:20 +02:00
parent 158a968bcc
commit e8a2fb924d
7 changed files with 49 additions and 44 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Riot/Modules/Spaces/SpaceList/SpaceListViewController.swift
+10 -24
View File
@@ -41,8 +41,6 @@ final class SpaceListViewController: UIViewController {
private var activityPresenter: ActivityIndicatorPresenter!
private var sections: [SpaceListSection] = []
private var homeIndexPath: IndexPath = IndexPath(row: 0, section: 0)
private var selectedIndexPath: IndexPath = IndexPath(row: 0, section: 0)
// MARK: - Setup
@@ -104,7 +102,6 @@ final class SpaceListViewController: UIViewController {
}
private func setupTableView() {
self.selectedIndexPath = self.homeIndexPath
self.tableView.separatorStyle = .none
self.tableView.rowHeight = UITableView.automaticDimension
self.tableView.estimatedRowHeight = Constants.estimatedRowHeight
@@ -117,8 +114,10 @@ final class SpaceListViewController: UIViewController {
switch viewState {
case .loading:
self.renderLoading()
case .loaded(let sections, let homeIndexPath):
self.renderLoaded(sections: sections, homeIndexPath: homeIndexPath)
case .loaded(let sections):
self.renderLoaded(sections: sections)
case .selectionChanged(let indexPath):
self.renderSelectionChanged(at: indexPath)
case .error(let error):
self.render(error: error)
}
@@ -131,16 +130,14 @@ final class SpaceListViewController: UIViewController {
}
}
private func renderLoaded(sections: [SpaceListSection], homeIndexPath: IndexPath) {
let isHomeSelected = self.selectedIndexPath.section == self.homeIndexPath.section
private func renderLoaded(sections: [SpaceListSection]) {
self.activityPresenter.removeCurrentActivityIndicator(animated: true)
self.sections = sections
self.tableView.reloadData()
self.homeIndexPath = homeIndexPath
if isHomeSelected {
self.selectedIndexPath = self.homeIndexPath
}
self.tableView.selectRow(at: selectedIndexPath, animated: true, scrollPosition: .none)
}
private func renderSelectionChanged(at indexPath: IndexPath) {
self.tableView.selectRow(at: indexPath, animated: true, scrollPosition: .none)
}
private func render(error: Error) {
@@ -155,15 +152,6 @@ extension SpaceListViewController: SpaceListViewModelViewDelegate {
func spaceListViewModel(_ viewModel: SpaceListViewModelType, didUpdateViewState viewSate: SpaceListViewState) {
self.render(viewState: viewSate)
}
func spaceListViewModel(_ viewModel: SpaceListViewModelType, didSelectSpaceAt indexPath: IndexPath) {
self.selectedIndexPath = IndexPath(row: indexPath.row, section: indexPath.section)
self.tableView.selectRow(at: indexPath, animated: true, scrollPosition: .none)
}
func spaceListViewModelRevertSelection(_ viewModel: SpaceListViewModelType) {
self.tableView.selectRow(at: self.selectedIndexPath, animated: true, scrollPosition: .none)
}
}
// MARK: - UITableViewDataSource
@@ -216,9 +204,7 @@ extension SpaceListViewController: UITableViewDataSource {
extension SpaceListViewController: UITableViewDelegate {
func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
if selectedIndexPath != indexPath {
self.viewModel.process(viewAction: .selectRow(at: indexPath, from: tableView.cellForRow(at: indexPath)))
}
self.viewModel.process(viewAction: .selectRow(at: indexPath, from: tableView.cellForRow(at: indexPath)))
}
}