feat: add console warning for misused ion-menu contentId

[willmartian]

Prequisites

• I have read the Contributing Guidelines.
• I agree to follow the Code of Conduct.
• I have searched for existing issues that already include this feature request, without success.

Describe the Feature Request

Add a console warning for when the contentId of an ion-menu is set to the id of the menu's inner ion-content.

Describe the Use Case

The contentId property is frequently misapplied. This is already mentioned in the docs (here); adding a console warning might help prevent this issue.

Describe Preferred Solution

When the inner ion-content's id is equal to its parent's contentId, we should trigger a console.warn.

An example of a warning in another component can be seen here:

ionic-framework/core/src/components/split-pane/split-pane.tsx

Line 142 in 7315e01

console.warn('split pane cannot have more than one main node');

Describe Alternatives

No response

Related Code

The following code snippet is a common error that will result in the ion-menu not behaving correctly.

<ion-menu contentId="my-content">
  <ion-content id="my-content"></ion-content>
</ion-menu>

Additional Information

No response

[ionitron-bot]

This issue has been labeled as help wanted. This label is added to issues that we believe would be good for contributors.

If you'd like to work on this issue, please comment here letting us know that you would like to submit a pull request for it. This helps us to keep track of the pull request and make sure there isn't duplicated effort.

For a guide on how to create a pull request and test this project locally to see your changes, see our contributing documentation.

Thank you!

[willmartian]

Resolved in v6 of Framework :)

[ionitron-bot]

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.