Adjust version validation script

Author: danielsaidi

Sources/SwiftUIKit/Lists/ListAction.swift

@@ -29,6 +29,16 @@ public enum ListAction {
 
 public extension ListAction {
 
+    var accessibilityLabel: String {
+        switch self {
+        case .call(let url): "Call \(url)"
+        case .copy(let text): "Copy \(text.prefix(10))"
+        case .copyImage: "Copy Image"
+        case .email(let addr): "Email \(addr)"
+        case .openUrl(let url): "Open \(url)"
+        }
+    }
+    
     @ViewBuilder
     var button: some View {
         button {
@@ -41,16 +51,11 @@ public extension ListAction {
         @ViewBuilder content: @escaping () -> Content
     ) -> some View {
         switch self {
-        case .call(let url):
-            link(url: .init(string: "tel:\(url)"), content: content)
-        case .copy(let text):
-            button(action: { copy(text) }, content: content)
-        case .copyImage(let image):
-            button(action: { copy(image) }, content: content)
-        case .email(let url):
-            link(url: .init(string: "mailto:\(url)"), content: content)
-        case .openUrl(let url):
-            link(url: .init(string: url), content: content)
+        case .call(let url): link(for: calllUrl(for: url), content)
+        case .copy(let text): button({ copy(text) }, content: content)
+        case .copyImage(let img): button({ copy(img) }, content: content)
+        case .email(let addr): link(for: emailUrl(for: addr), content)
+        case .openUrl(let str): link(for: url(for: str), content)
         }
     }
 
@@ -64,21 +69,37 @@ public extension ListAction {
     }
 }
 
+private extension ListAction {
+    
+    func emailUrl(for url: String) -> URL? {
+        .init(string: "mailto:\(url)")
+    }
+    
+    func calllUrl(for url: String) -> URL? {
+        .init(string: "tel:\(url)")
+    }
+    
+    func url(for url: String) -> URL? {
+        .init(string: url)
+    }
+}
+
 private extension ListAction {
 
     func button<Content: View>(
-        action: @escaping () -> Void,
+        _ action: @escaping () -> Void,
         @ViewBuilder content: @escaping () -> Content
     ) -> some View {
         Button(action: action) {
             content()
         }
+        .accessibilityHint(Text(accessibilityLabel))
     }
 
     @ViewBuilder
     func link<Content: View>(
-        url: URL?, 
-        @ViewBuilder content: @escaping () -> Content
+        for url: URL?,
+        @ViewBuilder _ content: @escaping () -> Content
     ) -> some View {
         if let url {
             Link(destination: url) {

Sources/SwiftUIKit/Lists/ListActionRow.swift

@@ -9,21 +9,16 @@
 #if os(iOS)
 import SwiftUI
 
-/**
- This view can be used to present action rows in a `List`.
- 
- If `hideIfEmpty` is `true` and the text is empty, this view
- is rendered as an `EmptyView`.
- */
+/// This view can be used to present action rows in a `List`.
 public struct ListActionRow: View {
 
     /// Create a list action row with a custom trailing view.
     ///
     /// - Parameters:
     ///   - title: The row title, if any.
     ///   - text: The row text.
-    ///   - action: The ``ListAction`` to use.
-    ///   - trailingView: An optional trailing view to apply to the view.
+    ///   - bundle: The bundle to use for localization, if any.
+    ///   - action: The ``ListAction`` to use, if any.
     public init(
         title: LocalizedStringKey? = nil,
         text: LocalizedStringKey,
@@ -80,9 +75,9 @@ public struct ListActionRow: View {
 
         ListActionRow(
             text: "Preview.Text.\(2) Preview.Text.\(2) Preview.Text.\(2) Preview.Text.\(2) Preview.Text.\(2)  Preview.Text.\(2)  Preview.Text.\(2) ",
-            bundle: .module
+            bundle: .module,
+            action: .email(address: "")
         )
-        .font(.footnote)
 
         ListActionRow(
             title: "Preview.Title.\(3)",

scripts/version_validate_git.sh

@@ -33,7 +33,7 @@ if ! git rev-parse --is-inside-work-tree > /dev/null 2>&1; then
 fi
 
 # Check for uncommitted changes
-if ! git diff-index --quiet HEAD --; then
+if [ -n "$(git status --porcelain)" ]; then
     echo "Error: Git repository is dirty. There are uncommitted changes."
     exit 1
 fi