Skip to content

Commit 0a9d1d9

Browse files
[Building Sync-ups Tutorial] fixes in "Sync-up form" chapter (#3576)
* Update ThemePicker to use foregroundStyle instead of deprecated foregroundColor * Fix typos in tutorial documentation Corrected minor grammatical and spelling errors in SyncUpForm and TestingSyncUpForm tutorial files * Remove duplicate TestStore initialization in tutorial code * Update tutorial documentation to use non-exhaustive testing method Corrected the wording in the tutorial step description for testing the "Add sync-up" flow to specify a non-exhaustive testing approach * Update Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/04-PresentingSyncUpForm/TestingSyncUpFormPresentation.tutorial --------- Co-authored-by: Stephen Celis <[email protected]>
1 parent a1a714b commit 0a9d1d9

12 files changed

+12
-13
lines changed

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm-01-code-0009.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ struct ThemePicker: View {
5454
Label(theme.name, systemImage: "paintpalette")
5555
.padding(4)
5656
}
57-
.foregroundColor(theme.accentColor)
57+
.foregroundStyle(theme.accentColor)
5858
.fixedSize(horizontal: false, vertical: true)
5959
.tag(theme)
6060
}

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm-02-code-0001-previous.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ struct ThemePicker: View {
5454
Label(theme.name, systemImage: "paintpalette")
5555
.padding(4)
5656
}
57-
.foregroundColor(theme.accentColor)
57+
.foregroundStyle(theme.accentColor)
5858
.fixedSize(horizontal: false, vertical: true)
5959
.tag(theme)
6060
}

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm-02-code-0001.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ struct ThemePicker: View {
6060
Label(theme.name, systemImage: "paintpalette")
6161
.padding(4)
6262
}
63-
.foregroundColor(theme.accentColor)
63+
.foregroundStyle(theme.accentColor)
6464
.fixedSize(horizontal: false, vertical: true)
6565
.tag(theme)
6666
}

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm-02-code-0002.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ struct ThemePicker: View {
6262
Label(theme.name, systemImage: "paintpalette")
6363
.padding(4)
6464
}
65-
.foregroundColor(theme.accentColor)
65+
.foregroundStyle(theme.accentColor)
6666
.fixedSize(horizontal: false, vertical: true)
6767
.tag(theme)
6868
}

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm-02-code-0003.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ struct ThemePicker: View {
6363
Label(theme.name, systemImage: "paintpalette")
6464
.padding(4)
6565
}
66-
.foregroundColor(theme.accentColor)
66+
.foregroundStyle(theme.accentColor)
6767
.fixedSize(horizontal: false, vertical: true)
6868
.tag(theme)
6969
}

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm-02-code-0004.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ struct ThemePicker: View {
6464
Label(theme.name, systemImage: "paintpalette")
6565
.padding(4)
6666
}
67-
.foregroundColor(theme.accentColor)
67+
.foregroundStyle(theme.accentColor)
6868
.fixedSize(horizontal: false, vertical: true)
6969
.tag(theme)
7070
}

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm-02-code-0005.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ struct ThemePicker: View {
7070
Label(theme.name, systemImage: "paintpalette")
7171
.padding(4)
7272
}
73-
.foregroundColor(theme.accentColor)
73+
.foregroundStyle(theme.accentColor)
7474
.fixedSize(horizontal: false, vertical: true)
7575
.tag(theme)
7676
}

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm-02-code-0006.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ struct ThemePicker: View {
7070
Label(theme.name, systemImage: "paintpalette")
7171
.padding(4)
7272
}
73-
.foregroundColor(theme.accentColor)
73+
.foregroundStyle(theme.accentColor)
7474
.fixedSize(horizontal: false, vertical: true)
7575
.tag(theme)
7676
}

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/SyncUpForm.tutorial

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,7 @@
254254

255255
@Step {
256256
Implement the logic of focusing a new added attendee. Now that we have immediate access
257-
to the reducer's logic we no longer to do any force unwrapping or defensive programming
257+
to the reducer's logic we no longer have to do any force unwrapping or defensive programming
258258
with `guard`s.
259259

260260
@Code(name: "SyncUpForm.swift", file: SyncUpForm-03-code-0002.swift)

Sources/ComposableArchitecture/Documentation.docc/Tutorials/BuildingSyncUps/03-SyncUpForm/TestingSyncUpForm.tutorial

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,10 @@
4343

4444
@Step {
4545
Add a new test method for testing what happens when we delete a focused attendee. This time
46-
the initial state with start with two attendees, which have been pulled out into variables
46+
the initial state will start with two attendees, which have been pulled out into variables
4747
so that we can reference them. The first attendee will also be focused.
4848

49-
> Note: We have collapsed the `testRemoveAttendee` method in the code snippet.
49+
> Note: We have collapsed the `removeAttendee` test method in the code snippet.
5050

5151
@Code(name: "SyncUpFormTests.swift", file: TestingSyncUpForm-01-code-0004.swift, previousFile: TestingSyncUpForm-01-code-0004-previous.swift)
5252
}

0 commit comments

Comments
 (0)