fix: use concrete tertiary color for row chevrons inside tinted buttons
This commit is contained in:
@@ -102,8 +102,9 @@ Gotchas learned the hard way:
|
|||||||
.combine)` (or a Button/control) to be queryable.
|
.combine)` (or a Button/control) to be queryable.
|
||||||
- List/Form section headers render uppercased unless `.textCase(nil)` —
|
- List/Form section headers render uppercased unless `.textCase(nil)` —
|
||||||
tests assert exact header strings.
|
tests assert exact header strings.
|
||||||
- Inside tinted Button rows, hierarchical `.primary`/`.secondary` foreground
|
- Inside tinted Button rows, hierarchical `.primary`/`.secondary`/`.tertiary`
|
||||||
styles resolve to the tint; use concrete `Color.primary`/`Color.secondary`.
|
foreground styles resolve to the tint (e.g. blue chevrons); use concrete
|
||||||
|
`Color.primary`/`Color.secondary`/`Color(.tertiaryLabel)`.
|
||||||
- The unblock confirmation dialog is queried via `app.sheets.buttons[...]`
|
- The unblock confirmation dialog is queried via `app.sheets.buttons[...]`
|
||||||
(a bare `buttons["Unblock"]` is ambiguous with the row label).
|
(a bare `buttons["Unblock"]` is ambiguous with the row label).
|
||||||
|
|
||||||
|
|||||||
@@ -80,7 +80,7 @@ struct NewRuleSheet: View {
|
|||||||
Spacer()
|
Spacer()
|
||||||
Image(systemName: "chevron.right")
|
Image(systemName: "chevron.right")
|
||||||
.font(.caption.weight(.semibold))
|
.font(.caption.weight(.semibold))
|
||||||
.foregroundStyle(.tertiary)
|
.foregroundStyle(Color(.tertiaryLabel))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
.accessibilityIdentifier("ruleKind-\(kind.rawValue)")
|
.accessibilityIdentifier("ruleKind-\(kind.rawValue)")
|
||||||
|
|||||||
@@ -216,7 +216,7 @@ struct RuleEditorView: View {
|
|||||||
.foregroundStyle(Color.secondary)
|
.foregroundStyle(Color.secondary)
|
||||||
Image(systemName: "chevron.right")
|
Image(systemName: "chevron.right")
|
||||||
.font(.caption.weight(.semibold))
|
.font(.caption.weight(.semibold))
|
||||||
.foregroundStyle(.tertiary)
|
.foregroundStyle(Color(.tertiaryLabel))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
.accessibilityIdentifier("selectedAppsRow")
|
.accessibilityIdentifier("selectedAppsRow")
|
||||||
|
|||||||
Reference in New Issue
Block a user