Merge branch 'primefaces:master' into master

gucal · Feb 13, 2024 · cecfa3b · cecfa3b
2 parents 5c465e0 + 13e0879
commit cecfa3b
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 14 deletions.
diff --git a/.github/ISSUE_TEMPLATE/bug_report.yml b/.github/ISSUE_TEMPLATE/bug_report.yml
@@ -20,9 +20,9 @@ body:
       id: reproducer
       attributes:
           label: Reproducer
-          placeholder: https://codesandbox.io/s/primereact-test-forked-bbns8k
+          placeholder: https://stackblitz.com/edit/vitejs-vite-daiajq
           description: |
-              Please fork the [CodeSandbox project](https://codesandbox.io/s/primereact-test-forked-bbns8k) or [Stackblitz project](https://stackblitz.com/edit/vitejs-vite-daiajq?file=src%2FApp.tsx) and create a case demonstrating your bug report. Issues **without** a CodeSandbox have much less possibility to be reviewed.
+              Please fork [Stackblitz project](https://stackblitz.com/edit/vitejs-vite-daiajq?file=src%2FApp.tsx) and create a case demonstrating your bug report. Issues **without** a CodeSandbox have much less possibility to be reviewed.
       validations:
           required: false
     - type: input

diff --git a/.github/workflows/issue-labeled.yml b/.github/workflows/issue-labeled.yml
@@ -31,4 +31,4 @@ jobs:
           token: ${{ secrets.GITHUB_TOKEN }}
           issue-number: ${{ github.event.issue.number }}
           body: |
-            Please fork the [CodeSandbox project](https://codesandbox.io/s/primereact-test-forked-bbns8k) or [Stackblitz project](https://stackblitz.com/edit/vitejs-vite-daiajq?file=src%2FApp.tsx) and create a case demonstrating your bug report. This issue will be closed if no activities in 20 days.
+            Please fork the [Stackblitz project](https://stackblitz.com/edit/vitejs-vite-daiajq?file=src%2FApp.tsx) and create a case demonstrating your bug report. This issue will be closed if no activities in 20 days.
diff --git a/components/lib/menubar/Menubar.js b/components/lib/menubar/Menubar.js
@@ -356,7 +356,7 @@ export const Menubar = React.memo(
 
         const onEnterKey = (event) => {
             if (focusedItemInfo.index !== -1) {
-                const element = DomHandler.findSingle(rootMenuRef.current, `li[id="${`${focusedItemId}`}"]`);
+                const element = DomHandler.findSingle(rootMenuRef.current, `li[data-id="${`${focusedItemId}`}"]`);
                 const anchorElement = element && DomHandler.findSingle(element, 'a[data-pc-section="action"]');
 
                 anchorElement ? anchorElement.click() : element && element.click();
@@ -483,7 +483,7 @@ export const Menubar = React.memo(
 
         const scrollInView = (index = -1) => {
             const id = index !== -1 ? `${idState}_${index}` : focusedItemId;
-            const element = DomHandler.findSingle(rootMenuRef.current, `li[id="${id}"]`);
+            const element = DomHandler.findSingle(rootMenuRef.current, `li[data-id="${id}"]`);
 
             if (element) {
                 element.scrollIntoView && element.scrollIntoView({ block: 'nearest', inline: 'start' });

diff --git a/components/lib/menubar/MenubarSub.js b/components/lib/menubar/MenubarSub.js
@@ -63,10 +63,10 @@ export const MenubarSub = React.memo(
         };
 
         const getItemId = (processedItem) => {
-            if (processedItem.item && processedItem.item.id) {
-                return processedItem.item.id;
-            }
+            return processedItem.item?.id;
+        };
 
+        const getItemDataId = (processedItem) => {
             return `${props.id}_${processedItem.key}`;
         };
 
@@ -87,7 +87,7 @@ export const MenubarSub = React.memo(
         };
 
         const isItemFocused = (processedItem) => {
-            return props.focusedItemId === getItemId(processedItem);
+            return props.focusedItemId === getItemDataId(processedItem);
         };
 
         const isItemGroup = (processedItem) => {
@@ -106,8 +106,8 @@ export const MenubarSub = React.memo(
             const key = props.id + '_separator_' + index;
             const separatorProps = mergeProps(
                 {
-                    id: key,
                     key,
+                    'data-id': key,
                     className: cx('separator'),
                     role: 'separator'
                 },
@@ -150,7 +150,8 @@ export const MenubarSub = React.memo(
                 return null;
             }
 
-            const key = getItemId(processedItem);
+            const id = getItemId(processedItem);
+            const dataId = getItemDataId(processedItem);
             const active = isItemActive(processedItem);
             const focused = isItemFocused(processedItem);
             const disabled = isItemDisabled(processedItem) || false;
@@ -236,8 +237,9 @@ export const MenubarSub = React.memo(
 
             const menuitemProps = mergeProps(
                 {
-                    id: key,
-                    key,
+                    id,
+                    key: id,
+                    'data-id': dataId,
                     role: 'menuitem',
                     'aria-label': item.label,
                     'aria-disabled': disabled,
@@ -283,7 +285,7 @@ export const MenubarSub = React.memo(
                 onFocus: props.onFocus,
                 onBlur: props.onBlur,
                 onKeyDown: props.onKeyDown,
-                id: props.id,
+                'data-id': props.id,
                 tabIndex: tabIndex,
                 'aria-activedescendant': props.ariaActivedescendant,
                 style: props.style,