Skip to content

Add TabContentFlow for iOS 18+ Tab API - #57

Merged
janpollak merged 3 commits into
mainfrom
feature/tab-content-flow
May 12, 2026
Merged

janpollak merged 3 commits into
mainfrom
feature/tab-content-flow

Conversation

@janpollak

Copy link
Copy Markdown
Contributor

Summary

  • Přidává TabContentFlow — varianta TabViewFlow pro iOS 18+ Tab API (@TabContentBuilder)
  • Umožňuje použití Tab(role: .search) a dalších iOS 18+ tab features v coordinator patternu
  • Extrahuje sdílenou modal logiku do ModalCoverModifier (snížení duplikace)
  • Non-breaking: stávající TabViewFlow zůstává pro zpětnou kompatibilitu

@janpollak
janpollak requested review from jmarek41, matejsemancik and ssestak and removed request for matejsemancik March 23, 2026 12:19
Comment thread Sources/FuturedArchitecture/Architecture/TabViewFlow.swift Outdated
Comment on lines +46 to +47
@available(iOS 18.0, macOS 15.0, tvOS 18.0, watchOS 11.0, *)
public struct TabContentFlow<Coordinator: TabCoordinator, Content: TabContent<Coordinator.Tab>>: View {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zaroven by som pridal deprecation flag na tú pôvodnú štruktúru TabViewFlow

@janpollak
janpollak force-pushed the feature/tab-content-flow branch from e3d9ba9 to 65e3aa8 Compare March 25, 2026 14:01
@jmarek41

jmarek41 commented Apr 2, 2026

Copy link
Copy Markdown
Member

@janpollak mrkni na ty Šimonovo připomínky prosím ať to máte ready na next projekt

janpollak added 3 commits May 12, 2026 12:48
Enables use of Tab(role: .search) and other iOS 18+ tab features
within the FuturedArchitecture coordinator pattern. Extracts shared
modal presentation logic into ModalCoverModifier to reduce duplication.
Pulls the sheet/full-screen cover binding boilerplate out of TabViewFlow
into a standalone ModalCoverModifier generic over any Coordinator. Apply
it in NavigationStackFlow too so all flow containers share the same
modal presentation logic.
Uses per-platform availability so projects still targeting iOS 16/17
(or other pre-iOS 18 platforms) are not warned, while iOS 18+ targets
get a nudge towards TabContentFlow.
@janpollak
janpollak force-pushed the feature/tab-content-flow branch from 65e3aa8 to 49ec2bf Compare May 12, 2026 10:50
@janpollak

Copy link
Copy Markdown
Contributor Author

@ssestak, all fixed.

@janpollak
janpollak requested a review from ssestak May 12, 2026 10:56
@janpollak
janpollak merged commit 0725870 into main May 12, 2026
1 check passed
@janpollak
janpollak deleted the feature/tab-content-flow branch May 12, 2026 11:29
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

Successfully merging this pull request may close these issues.

3 participants