Skip to content

Conversation

ShaneK
Copy link
Member

@ShaneK ShaneK commented Sep 19, 2025

Issue number: resolves #30641


What is the current behavior?

Currently, Vue modals do not emit ionModal... events. This happened due to a change in the way the stencil output targets for Vue changed. Christian updated the overlays to support the base events, but not the component-specific events.

What is the new behavior?

With this change, you'll be able to bind to the events as described in the Ionic documentation.

Does this introduce a breaking change?

  • Yes
  • No

Other information

Current dev build:

8.7.5-dev.11758311583.14f4e9d9

@ShaneK ShaneK requested a review from a team as a code owner September 19, 2025 19:52
@ShaneK ShaneK requested a review from gnbm September 19, 2025 19:52
Copy link

vercel bot commented Sep 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
ionic-framework Ready Ready Preview Comment Sep 23, 2025 4:58pm

@github-actions github-actions bot added the package: vue @ionic/vue package label Sep 19, 2025
@ShaneK ShaneK changed the title fix(vue): add component-specific overlay events fix(vue): emit component-specific overlay events Sep 19, 2025
Copy link
Member

@brandyscarney brandyscarney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fix looks good! I mostly have minor suggestions and questions about whether we should add more tests or not. I will defer to you on if you think we need them. 👍

@ShaneK ShaneK added this pull request to the merge queue Sep 23, 2025
Merged via the queue into main with commit 024d090 Sep 23, 2025
51 checks passed
@ShaneK ShaneK deleted the FW-6758 branch September 23, 2025 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: vue @ionic/vue package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: Vue ion-modal ionModalDidPresent not emitted since 8.4.4 to latest (8.7.2)
2 participants