diff --git a/Sources/Compound/List/ListRowAccessory.swift b/Sources/Compound/List/ListRowAccessory.swift index 28a5bde..80ddbbb 100644 --- a/Sources/Compound/List/ListRowAccessory.swift +++ b/Sources/Compound/List/ListRowAccessory.swift @@ -72,13 +72,12 @@ public struct ListRowAccessory: View { .padding(.vertical, verticalPaddingFix) case .multiSelected: CompoundIcon(\.checkCircleSolid) - .foregroundColor(isEnabled ? .compound.iconPrimary : .compound.iconDisabled) + .foregroundColor(isEnabled ? .compound.iconSuccessPrimary : .compound.iconDisabled) .accessibilityAddTraits(.isSelected) .padding(.vertical, verticalPaddingFix) case .multiUnselected: - CompoundIcon(\.checkCircle) - .foregroundColor(isEnabled ? .compound.iconTertiary : .compound.iconDisabled) - .overlay { Circle().inset(by: circleOverlayInsets).fill(.background) } // This is a bodge. + CompoundIcon(\.circle) + .foregroundColor(isEnabled ? .compound.borderInteractivePrimary : .compound.borderDisabled) .padding(.vertical, verticalPaddingFix) } } @@ -122,16 +121,3 @@ struct ListRowAccessory_Previews: PreviewProvider, PrefireProvider { } } } - -/// Just to prove that the overlay bodge works for all Dynamic Type sizes. -struct MultiUnselectedAccessory_Previews: PreviewProvider, PrefireProvider { - static var previews: some View { - VStack { - ForEach(DynamicTypeSize.allCases, id: \.self) { size in - ListRowAccessory.multiSelection(false) - .dynamicTypeSize(size) - } - } - .previewDisplayName("Fake circle icon") - } -} diff --git a/Tests/CompoundTests/__Snapshots__ b/Tests/CompoundTests/__Snapshots__ index 7e4417e..c4080ee 160000 --- a/Tests/CompoundTests/__Snapshots__ +++ b/Tests/CompoundTests/__Snapshots__ @@ -1 +1 @@ -Subproject commit 7e4417e95c10eecbad01b60d77d0544b2bccb256 +Subproject commit c4080ee11ae3c43bd08a1a0d3efd8460df384000