From 1e6d9fd1662464e649539d06095336f23b0a44d6 Mon Sep 17 00:00:00 2001
From: Lukas Harbarth <lukas.harbarth@sap.com>
Date: Tue, 5 Nov 2024 17:40:52 +0100
Subject: [PATCH] fix(MessageViewButton): add default tooltip & aria-label

---
 .storybook/mockData/generateMessageItems.tsx  | 17 ----------
 .../MessageView/MessageView.stories.tsx       | 11 ------
 .../components/MessageViewButton/index.tsx    | 34 ++++++++++++++-----
 .../main/src/i18n/messagebundle.properties    | 12 +++++++
 4 files changed, 37 insertions(+), 37 deletions(-)

diff --git a/.storybook/mockData/generateMessageItems.tsx b/.storybook/mockData/generateMessageItems.tsx
index 0995607b9fc..1cbae6ff3f9 100644
--- a/.storybook/mockData/generateMessageItems.tsx
+++ b/.storybook/mockData/generateMessageItems.tsx
@@ -24,22 +24,6 @@ const informationMessageItems = (count) => {
   ));
 };
 
-const successMessageItems = (count) => {
-  if (!count) {
-    return [];
-  }
-  return new Array(count).fill('').map((_, index) => (
-    <MessageItem
-      key={`SuccessMessage${index}`}
-      titleText={'Success Message'}
-      type={ValueState.Success}
-      groupName={`Group${index}`}
-    >
-      {LoremIpsum}
-    </MessageItem>
-  ));
-};
-
 const warningMessageItems = (count) => {
   if (!count) {
     return [];
@@ -75,6 +59,5 @@ const errorMessageItems = (count) => {
 export const generateMessageItems = (numberOfMessageTypes) => [
   ...informationMessageItems(numberOfMessageTypes.information),
   ...warningMessageItems(numberOfMessageTypes.warning),
-  ...successMessageItems(numberOfMessageTypes.success),
   ...errorMessageItems(numberOfMessageTypes.error)
 ];
diff --git a/packages/main/src/components/MessageView/MessageView.stories.tsx b/packages/main/src/components/MessageView/MessageView.stories.tsx
index 196ad5fcec3..18f43be6a4d 100644
--- a/packages/main/src/components/MessageView/MessageView.stories.tsx
+++ b/packages/main/src/components/MessageView/MessageView.stories.tsx
@@ -13,7 +13,6 @@ import type {
 import { Bar } from '../../webComponents/Bar/index.js';
 import { Button } from '../../webComponents/Button/index.js';
 import { Dialog as OriginalDialog } from '../../webComponents/Dialog';
-import { Link } from '../../webComponents/Link/index.js';
 import { ResponsivePopover as OriginalResponsivePopover } from '../../webComponents/ResponsivePopover';
 import { Title } from '../../webComponents/Title/index.js';
 import { FlexBox } from '../FlexBox/index.js';
@@ -60,16 +59,6 @@ const meta = {
       >
         First Error Message Description.
       </MessageItem>,
-      <MessageItem
-        key={2}
-        titleText={'Success Message Type'}
-        subtitleText={'You can also use subtitles'}
-        type={ValueState.Success}
-        counter={2}
-      >
-        This is a success message! You can even use{' '}
-        <Link href={'https://github.com/SAP/ui5-webcomponents-react'}>links here</Link>.
-      </MessageItem>,
       <MessageItem
         key={3}
         titleText={'Warning Message Type'}
diff --git a/packages/main/src/components/MessageViewButton/index.tsx b/packages/main/src/components/MessageViewButton/index.tsx
index 4dd10a1a54a..4f6bb78a3c9 100644
--- a/packages/main/src/components/MessageViewButton/index.tsx
+++ b/packages/main/src/components/MessageViewButton/index.tsx
@@ -4,10 +4,11 @@ import alertIcon from '@ui5/webcomponents-icons/dist/alert.js';
 import errorIcon from '@ui5/webcomponents-icons/dist/error.js';
 import informationIcon from '@ui5/webcomponents-icons/dist/information.js';
 import sysEnter2Icon from '@ui5/webcomponents-icons/dist/sys-enter-2.js';
-import { useStylesheet } from '@ui5/webcomponents-react-base';
+import { useI18nBundle, useStylesheet } from '@ui5/webcomponents-react-base';
 import { clsx } from 'clsx';
 import { forwardRef } from 'react';
 import { ValueState } from '../../enums/index.js';
+import { ERROR_TYPE, WARNING_TYPE, INFORMATION_TYPE, SUCCESS_TYPE } from '../../i18n/i18n-defaults.js';
 import type { ButtonDomRef, ButtonPropTypes } from '../../webComponents/index.js';
 import { Button } from '../../webComponents/index.js';
 import { classNames, styleData } from './MessageViewButton.module.css.js';
@@ -28,16 +29,21 @@ export interface MessageViewButtonProptypes
   counter?: number;
 }
 
-const getIcon = (type) => {
+interface Types {
+  icon: string;
+  i18nLabel: { key: string; defaultText: string };
+}
+
+const getTypes = (type: MessageViewButtonProptypes['type']): Types => {
   switch (type) {
     case ValueState.Error:
-      return errorIcon;
+      return { icon: errorIcon, i18nLabel: ERROR_TYPE };
     case ValueState.Success:
-      return sysEnter2Icon;
+      return { icon: sysEnter2Icon, i18nLabel: WARNING_TYPE };
     case ValueState.Warning:
-      return alertIcon;
+      return { icon: alertIcon, i18nLabel: INFORMATION_TYPE };
     default:
-      return informationIcon;
+      return { icon: informationIcon, i18nLabel: SUCCESS_TYPE };
   }
 };
 
@@ -45,13 +51,23 @@ const getIcon = (type) => {
  * The `MessageViewButton` can be used for opening a `Popover` containing the `MessageView` component. It should always reflect the message `type` with the highest severity.
  */
 const MessageViewButton = forwardRef<ButtonDomRef, MessageViewButtonProptypes>((props, ref) => {
-  const { type = ValueState.Error, counter, className, ...rest } = props;
+  const { type = ValueState.Error, counter, className, tooltip, accessibleName, ...rest } = props;
   useStylesheet(styleData, MessageViewButton.displayName);
   const classes = clsx(classNames.btn, className);
-  const icon = getIcon(type);
+  const { icon, i18nLabel } = getTypes(type);
 
+  const i18nBundle = useI18nBundle('@ui5/webcomponents-react');
+  const label = i18nBundle.getText(i18nLabel);
   return (
-    <Button ref={ref} className={classes} icon={icon} {...rest} data-type={type}>
+    <Button
+      ref={ref}
+      className={classes}
+      icon={icon}
+      {...rest}
+      data-type={type}
+      tooltip={tooltip ?? label}
+      accessibleName={accessibleName ?? label}
+    >
       {counter > 1 && counter}
     </Button>
   );
diff --git a/packages/main/src/i18n/messagebundle.properties b/packages/main/src/i18n/messagebundle.properties
index 0412c448448..a783350dc19 100644
--- a/packages/main/src/i18n/messagebundle.properties
+++ b/packages/main/src/i18n/messagebundle.properties
@@ -339,3 +339,15 @@ NO_DATA_FILTERED=No data found. Try adjusting the filter settings.
 #XACT: Number of items in a pseudo list
 WITH_X_ITEMS=with {0} items
 
+#XACT: Message View error button label
+ERROR_TYPE=Error Type
+
+#XACT: Message View warning button label
+WARNING_TYPE=Warning Type
+
+#XACT: Message View information button label
+INFORMATION_TYPE=Information Type
+
+#XACT: Message View success button label
+SUCCESS_TYPE=Success Type
+