CQI #13742 Step aria-controls showcase

[GiacomoDM]

Fix #13742 (doesn't actually fix, it's just for correct liking with the issue)

This PR showcase the usage of the MenuItem's new property overrideAriaControls. This allows to pass the MenuItem the id of the element that the step controls, as shown by the routing example in the Step component doc.

This is just an example and it's difficult to interact with it, because of the previous problems with the Step component that are to be fixed with #13743.

If this solution will be accepted, it will require a porting (or at least an evauation to decide if it's needed) to all other components that uses the MenuItem and have an aria-controls attribute.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
primeng-ssr-test	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 12, 2023 7:45am

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
primeng	⬜️ Ignored (Inspect)	Visit Preview		Oct 12, 2023 7:45am

[cetincakiroglu]

Hi,

New prop added to the menuItem api and not all menu components use MenuItem API can use this new feature. As I see it's specific to steps component, is it possible to find another way to implement it within the scope of the steps component? Because MenuItem API is a common interface and used by various components.

[GiacomoDM]

Hi @cetincakiroglu,
I have updated the PR. I extended MenuItem in a new interface StepItem, that only adds the aria property. This way it will always inherit any changes made to the MenuItem and work as expected.
I also updated the docs.

Let me know if this solution is acceptable for you.

[cetincakiroglu]

Hi @GiacomoDM,

Thanks a lot for the effort and support!

Could you please update the PR and resolve the conflicts so we can merge safely?