feat(progress-indicator): add semantics label for status of progress indicator

Author: nouha06

ouds_core/lib/components/progress_indicator/internal/ouds_progress_indicator_utils.dart

@@ -14,7 +14,9 @@
 library;
 
 import 'package:flutter/material.dart';
+import 'package:ouds_core/components/common/ouds_icon_status.dart';
 import 'package:ouds_core/components/progress_indicator/ouds_progress_indicator.dart';
+import 'package:ouds_core/l10n/gen/ouds_localizations.dart';
 
 /// Utility methods used by OUDS progress indicators.
 ///
@@ -76,4 +78,17 @@ class OudsProgressIndicatorUtils {
 
     return '';
   }
+
+  static String? buildStatusSemanticsLabel(
+    OudsLocalizations? localizations,
+    OudsIconStatus status,
+  ) {
+    return status is Warning
+        ? localizations?.core_common_warning_a11y
+        : status is Negative
+        ? localizations?.core_common_error_a11y
+        : status is Info
+        ? localizations?.core_common_info_a11y
+        : null;
+  }
 }

ouds_core/lib/components/progress_indicator/ouds_progress_indicator.dart

@@ -17,6 +17,7 @@ import 'package:ouds_core/components/common/ouds_icon_status.dart';
 import 'package:ouds_core/components/progress_indicator/internal/ouds_progress_indicator_status_modifier.dart';
 import 'package:ouds_core/components/progress_indicator/internal/ouds_progress_indicator_style_modifier.dart';
 import 'package:ouds_core/components/progress_indicator/internal/ouds_progress_indicator_utils.dart';
+import 'package:ouds_core/l10n/gen/ouds_localizations.dart';
 import 'package:ouds_theme_contract/ouds_theme.dart';
 
 /// Defines whether a progress indicator shows a known progress value
@@ -196,8 +197,12 @@ class _OudsCircularProgressIndicatorState
     required double gapSize,
     required StrokeCap strokeCap,
   }) {
+    final localizations = OudsLocalizations.of(context);
+    final semanticsLabel =
+        '${widget.semanticLabel} , ${OudsProgressIndicatorUtils.buildStatusSemanticsLabel(localizations, widget.status) ?? ''}, ';
+
     return CircularProgressIndicator(
-      semanticsLabel: widget.semanticLabel,
+      semanticsLabel: semanticsLabel,
       year2023: false,
       constraints: BoxConstraints(
         minWidth: defaultSize,
@@ -299,33 +304,39 @@ class _OudsLinearProgressIndicatorState
 
   @override
   Widget build(BuildContext context) {
-    return Column(
-      spacing: OudsTheme.of(
-        context,
-      ).componentsTokens(context).progressIndicator.spacePaddingBlock,
-      children: [
-        _buildLinearProgressIndicator(),
-        Align(
-          alignment: OudsProgressIndicatorUtils.getTextAlign(
-            widget.helperTextAlignment,
-          ),
-          child: Text(
-            OudsProgressIndicatorUtils.buildHelperText(
-              widget.percentage,
-              widget.spaceBeforePercentage,
-              widget.progress,
-              widget.helperText,
+    return MergeSemantics(
+      child: Column(
+        spacing: OudsTheme.of(
+          context,
+        ).componentsTokens(context).progressIndicator.spacePaddingBlock,
+        children: [
+          _buildLinearProgressIndicator(),
+          Align(
+            alignment: OudsProgressIndicatorUtils.getTextAlign(
+              widget.helperTextAlignment,
             ),
-            style: OudsTheme.of(context).typographyTokens
-                .typeLabelDefaultMedium(context)
-                .copyWith(
-                  color: OudsTheme.of(
-                    context,
-                  ).colorScheme(context).contentDefault,
-                ),
+            child: widget.percentage
+                ? ExcludeSemantics(child: _buildHelperTextWidget())
+                : _buildHelperTextWidget(),
+          ),
+        ],
+      ),
+    );
+  }
+
+  Widget _buildHelperTextWidget() {
+    return Text(
+      OudsProgressIndicatorUtils.buildHelperText(
+        widget.percentage,
+        widget.spaceBeforePercentage,
+        widget.progress,
+        widget.helperText,
+      ),
+      style: OudsTheme.of(context).typographyTokens
+          .typeLabelDefaultMedium(context)
+          .copyWith(
+            color: OudsTheme.of(context).colorScheme(context).contentDefault,
           ),
-        ),
-      ],
     );
   }
 
@@ -410,9 +421,14 @@ class _OudsLinearProgressIndicatorState
     required double gapSize,
     required BorderRadiusGeometry borderRadius,
   }) {
+    final localizations = OudsLocalizations.of(context);
+
+    final semanticsLabel =
+        '${widget.semanticLabel} , ${OudsProgressIndicatorUtils.buildStatusSemanticsLabel(localizations, widget.status) ?? ''}, ';
+
     return LinearProgressIndicator(
       minHeight: minHeight,
-      semanticsLabel: widget.semanticLabel,
+      semanticsLabel: semanticsLabel,
       year2023: false,
       value: value,
       valueColor: AlwaysStoppedAnimation<Color>(indicatorColor),

ouds_core/lib/l10n/gen/ouds_localizations.dart

@@ -115,6 +115,12 @@ abstract class OudsLocalizations {
   /// **'Warning'**
   String get core_common_warning_a11y;
 
+  /// No description provided for @core_common_info_a11y.
+  ///
+  /// In en, this message translates to:
+  /// **'Information'**
+  String get core_common_info_a11y;
+
   /// No description provided for @core_common_selected_a11y.
   ///
   /// In en, this message translates to:

ouds_core/lib/l10n/gen/ouds_localizations_ar.dart

@@ -17,6 +17,9 @@ class OudsLocalizationsAr extends OudsLocalizations {
   @override
   String get core_common_warning_a11y => 'تحذير';
 
+  @override
+  String get core_common_info_a11y => 'معلومة';
+
   @override
   String get core_common_selected_a11y => 'محدد';
 

ouds_core/lib/l10n/gen/ouds_localizations_en.dart

@@ -17,6 +17,9 @@ class OudsLocalizationsEn extends OudsLocalizations {
   @override
   String get core_common_warning_a11y => 'Warning';
 
+  @override
+  String get core_common_info_a11y => 'Information';
+
   @override
   String get core_common_selected_a11y => 'Selected';
 

ouds_core/lib/l10n/gen/ouds_localizations_fr.dart

@@ -17,6 +17,9 @@ class OudsLocalizationsFr extends OudsLocalizations {
   @override
   String get core_common_warning_a11y => 'Avertissement';
 
+  @override
+  String get core_common_info_a11y => 'Information';
+
   @override
   String get core_common_selected_a11y => 'Sélectionné';
 

ouds_core/lib/l10n/ouds_flutter_ar.arb

@@ -2,6 +2,7 @@
     "@_OUDS_COMMON": {},
     "core_common_error_a11y": "خطأ",
     "core_common_warning_a11y": "تحذير",
+    "core_common_info_a11y": "معلومة",
     "core_common_selected_a11y": "محدد",
     "core_common_unselected_a11y": "غير محدد",
     "core_common_loading_a11y": "جاري التحميل",

ouds_core/lib/l10n/ouds_flutter_en.arb

@@ -2,6 +2,7 @@
     "@_OUDS_COMMON": {},
     "core_common_error_a11y": "Error",
     "core_common_warning_a11y": "Warning",
+    "core_common_info_a11y": "Information",
     "core_common_selected_a11y": "Selected",
     "core_common_unselected_a11y": "Unselected",
     "core_common_loading_a11y": "Loading",

ouds_core/lib/l10n/ouds_flutter_fr.arb

@@ -2,7 +2,8 @@
     "@_OUDS_COMMON": {},
     "core_common_error_a11y": "Erreur",
     "core_common_warning_a11y": "Avertissement",
-    "core_common_selected_a11y": "Sélectionné",
+  "core_common_info_a11y": "Information",
+  "core_common_selected_a11y": "Sélectionné",
     "core_common_unselected_a11y": "Non sélectionné",
     "core_common_loading_a11y": "Chargement",
     "core_common_disabled_a11y": "Désactivé",