ReviewOS

also looking at this

craft-native/craft

feat: finish native customization and platform seams

#401
Merged glennmichael123 wants to merge codex/issues-388-400-344 into main
21 files +707 -17

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

packages/ios/src/index.test.tsmodified+1-0
Changes to packages/ios/src/index.test.ts
@@ -91,6 +91,7 @@ describe('Craft iOS builder', () => {
9191 expect(swift).toContain('private(set) var tabSelectionColor: UIColor?')
9292 expect(swift).toContain('appearance: CraftConfig.TabBar? = nil')
9393 expect(swift).toContain('chrome.tabSelectionColor.map { Color($0) }')
94 expect(swift).toContain('@ObservedObject private var chrome = CraftChrome.shared')
9495 expect(hybrid).toContain('appearance: config.tabBar')
9596 await expect(init({ name: 'Invalid Tab Bar', output: join(output, 'invalid'), runtimeDir: null, config: {
9697 tabBar: { selection: { cornerRadius: -1 } },
packages/ios/templates/CraftApp.swiftmodified+1-0
Changes to packages/ios/templates/CraftApp.swift
@@ -1321,6 +1321,7 @@ private struct CraftTabSelectionShape: Shape {
13211321}
13221322
13231323private struct CraftTabSelectionIndicator: View {
1324 @ObservedObject private var chrome = CraftChrome.shared
13241325 @ObservedObject private var progress = CraftChrome.shared.tabSwipeProgress
13251326 let selected: Int?
13261327 let count: Int