fix(modal, popover): overlays now automatically determine if they are inline

[liamdebeasi]

… being used inline or not

Pull request checklist

Please check if your PR fulfills the following requirements:

• Tests for the changes have been added (for bug fixes / features)
• Docs have been reviewed and added / updated if needed (for bug fixes / features)
• Build (npm run build) was run locally and any changes were pushed
• Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

• Bugfix
• Feature
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation content changes
• Other (please describe):

What is the current behavior?

Previously you needed to explicitly tell modal/popover that they were not being used inline which caused some breaking changes.

What is the new behavior?

• We can now automatically detect if the overlay is in the correct location (I.e. a direct descendant of ion-app or body) before presenting, and return the correct delegate based on that.

If overlay is in correct location, we just call present. Otherwise, we pass in either the value of this.delegate or the default core delegate.

Does this introduce a breaking change?

• Yes
• No

Other information