ui: tooltips for actions in extra files view

pixelspark · pixelspark · commit f931839dd086 · 2025-10-28T20:44:27.000+01:00
Signed-off-by: Tommy van der Vorst &lt;tommy@pixelspark.nl&gt;
diff --git a/Sushitrain/ExtraFilesView.swift b/Sushitrain/ExtraFilesView.swift
@@ -55,12 +55,12 @@ struct ExtraFilesView: View {
 										for f in extraFiles { verdicts[f] = s }
 									})
 							) {
-								Image(systemName: "trash").tint(.red).tag(false).accessibilityLabel("Delete file")
+								Image(systemName: "trash").tint(.red).tag(false).help("Delete file")
 								if folder.folderType() == SushitrainFolderTypeReceiveOnly {
-									Image(systemName: "trash.slash").tag(true).accessibilityLabel("Keep file")
+									Image(systemName: "trash.slash").tag(true).help("Keep file")
 								}
 								else {
-									Image(systemName: "plus.square.fill").tag(true).accessibilityLabel("Keep file")
+									Image(systemName: "plus.square.fill").tag(true).help("Keep file")
 								}
 							}.pickerStyle(.segmented).frame(width: 100)
 						}
@@ -92,16 +92,16 @@ struct ExtraFilesView: View {
 												allVerdict = nil
 											})
 									) {
-										Image(systemName: "trash").tint(.red).tag(false).accessibilityLabel("Delete file")
+										Image(systemName: "trash").tint(.red).tag(false).help("Delete file")
 										if folder.folderType() == SushitrainFolderTypeReceiveOnly {
-											Image(systemName: "trash.slash").tag(true).accessibilityLabel("Keep file")
+											Image(systemName: "trash.slash").tag(true).help("Keep file")
 										}
 										else {
 											if let ge = globalEntry, !ge.isDeleted() {
-												Image(systemName: "rectangle.2.swap").tag(true).accessibilityLabel("Replace existing file")
+												Image(systemName: "rectangle.2.swap").tag(true).help("Replace existing file")
 											}
 											else {
-												Image(systemName: "plus.square.fill").tag(true).accessibilityLabel("Keep file")
+												Image(systemName: "plus.square.fill").tag(true).help("Keep file")
 											}
 										}
 									}.pickerStyle(.segmented).frame(width: 100)

Original file line number	Diff line number	Diff line change
`@@ -55,12 +55,12 @@ struct ExtraFilesView: View {`
`55`	`55`	`for f in extraFiles { verdicts[f] = s }`
`56`	`56`	`})`
`57`	`57`	`) {`
`58`		`- Image(systemName: "trash").tint(.red).tag(false).accessibilityLabel("Delete file")`
	`58`	`+ Image(systemName: "trash").tint(.red).tag(false).help("Delete file")`
`59`	`59`	`if folder.folderType() == SushitrainFolderTypeReceiveOnly {`
`60`		`- Image(systemName: "trash.slash").tag(true).accessibilityLabel("Keep file")`
	`60`	`+ Image(systemName: "trash.slash").tag(true).help("Keep file")`
`61`	`61`	`}`
`62`	`62`	`else {`
`63`		`- Image(systemName: "plus.square.fill").tag(true).accessibilityLabel("Keep file")`
	`63`	`+ Image(systemName: "plus.square.fill").tag(true).help("Keep file")`
`64`	`64`	`}`
`65`	`65`	`}.pickerStyle(.segmented).frame(width: 100)`
`66`	`66`	`}`
`@@ -92,16 +92,16 @@ struct ExtraFilesView: View {`
`92`	`92`	`allVerdict = nil`
`93`	`93`	`})`
`94`	`94`	`) {`
`95`		`- Image(systemName: "trash").tint(.red).tag(false).accessibilityLabel("Delete file")`
	`95`	`+ Image(systemName: "trash").tint(.red).tag(false).help("Delete file")`
`96`	`96`	`if folder.folderType() == SushitrainFolderTypeReceiveOnly {`
`97`		`- Image(systemName: "trash.slash").tag(true).accessibilityLabel("Keep file")`
	`97`	`+ Image(systemName: "trash.slash").tag(true).help("Keep file")`
`98`	`98`	`}`
`99`	`99`	`else {`
`100`	`100`	`if let ge = globalEntry, !ge.isDeleted() {`
`101`		`- Image(systemName: "rectangle.2.swap").tag(true).accessibilityLabel("Replace existing file")`
	`101`	`+ Image(systemName: "rectangle.2.swap").tag(true).help("Replace existing file")`
`102`	`102`	`}`
`103`	`103`	`else {`
`104`		`- Image(systemName: "plus.square.fill").tag(true).accessibilityLabel("Keep file")`
	`104`	`+ Image(systemName: "plus.square.fill").tag(true).help("Keep file")`
`105`	`105`	`}`
`106`	`106`	`}`
`107`	`107`	`}.pickerStyle(.segmented).frame(width: 100)`