Skip to content

Commit af5c3ad

Browse files
committed
a11y: label config screen icon buttons and hide decorative glyphs for VoiceOver
1 parent e531f15 commit af5c3ad

2 files changed

Lines changed: 9 additions & 0 deletions

File tree

Meshtastic/Views/Settings/Config/LoRaConfig.swift

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,9 @@ struct LoRaConfig: View {
108108
HStack(alignment: .top, spacing: 8) {
109109
Image(systemName: licensed ? "checkmark.seal.fill" : "exclamationmark.triangle.fill")
110110
.foregroundColor(licensed ? .green : .orange)
111+
// Decorative status glyph; the adjacent "Licensed band" title and
112+
// description already convey licensed/unlicensed state to VoiceOver.
113+
.accessibilityHidden(true)
111114
VStack(alignment: .leading, spacing: 2) {
112115
Text("Licensed band")
113116
.font(.callout).bold()
@@ -226,6 +229,8 @@ struct LoRaConfig: View {
226229
HStack {
227230
Image(systemName: "antenna.radiowaves.left.and.right")
228231
.foregroundColor(.accentColor)
232+
// Decorative icon; the Stepper carries the label for VoiceOver.
233+
.accessibilityHidden(true)
229234
Stepper(txPower == 0 ? "Max Transmit Power" : "\(txPower)dBm Transmit Power", value: $txPower, in: 0...30, step: 1)
230235
.padding(5)
231236
}

Meshtastic/Views/Settings/Config/Module/AmbientLightingConfig.swift

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,16 @@ struct AmbientLightingConfig: View {
3737
HStack {
3838
Image(systemName: "eyedropper")
3939
.foregroundColor(.accentColor)
40+
// Decorative icon; the ColorPicker carries the label for VoiceOver.
41+
.accessibilityHidden(true)
4042
ColorPicker("Color", selection: $color, supportsOpacity: false)
4143
.padding(5)
4244
}
4345
HStack {
4446
Image(systemName: "directcurrent")
4547
.foregroundColor(.accentColor)
48+
// Decorative icon; the Stepper carries the label for VoiceOver.
49+
.accessibilityHidden(true)
4650
Stepper("Current: \(current)", value: $current, in: 0...31, step: 1)
4751
.padding(5)
4852
}

0 commit comments

Comments
 (0)