Repository navigation
bug: [angular] ModalController is not compatible with Angular signal inputs #30784
Closed as duplicate
Description
Prerequisites
- I have read the Contributing Guidelines.
- I agree to follow the Code of Conduct.
- I have searched for existing issues that already report this problem, without success.
Ionic Framework Version
v8.x
Current Behavior
ModalController is not compatible with Angular signal inputs
Expected Behavior
ModalController is compatible with Angular signal inputs
Steps to Reproduce
- Create Angular component with signal inputs
- Try to set its inputs using
ModalController.create().
Code Reproduction URL
Ionic Info
@ionic/angular: 8.7.9
Additional Information
No response
Activity
ionitron-bot commentedon Nov 18, 2025
Thanks for the issue! This issue has been labeled as
needs reproduction. This label is added to issues that need a code reproduction.Please reproduce this issue in an Ionic starter application and provide a way for us to access it (GitHub repo, StackBlitz, etc). Without a reliable code reproduction, it is unlikely we will be able to resolve the issue, leading to it being closed.
If you have already provided a code snippet and are seeing this message, it is likely that the code snippet was not enough for our team to reproduce the issue.
For a guide on how to create a good reproduction, see our Contributing Guide.
hakimio commentedon Nov 19, 2025
Ok, you have to manually enable
useSetInputAPIin Ionic config for this to work.@ShaneK I can't find this config in the docs and why can't Ionic just automatically check if the
setInput()method is defined?EDIT: this is duplicate of #28876 . Anyway,
useSetInputAPIshould be enabled by default at this point since it's available since Angularv14and@ionic/angularsupports only Angularv16+.ShaneK commentedon Nov 19, 2025
Excellent find, @hakimio! That's a solid point, we should probably do that. I wasn't sure what the root of this issue was, which is why I needed the reproduction, but I'm glad you found it. Thank you!
hakimio commentedon Nov 19, 2025
@ShaneK so, are you planning to enable this by default since all supported Angular versions provide
ComponentRef.setInput()API?ShaneK commentedon Nov 19, 2025
Yeah, I've created a card around validating our defaults for Angular apps with this specific point involved. It doesn't have priority right now and we're all pretty busy with other tasks, but at least we know about it now. If you'd like to PR the change, it would be pretty easy here:
https://github.com/ionic-team/starters
Specifically, these two places I believe:
https://github.com/ionic-team/starters/blob/main/angular-standalone/base
https://github.com/ionic-team/starters/blob/main/angular/base
hakimio commentedon Nov 19, 2025
The right thing to do would be to remove the config entirely and do the right thing by default instead of changing the starters to have this config enabled.
ShaneK commentedon Nov 19, 2025
Feel free to open a PR for that instead then. That's probably the conclusion we'll come to when we review it, but reviewing the defaults is still something that should be done.
hakimio commentedon Nov 21, 2025
@ShaneK I have opened #30803 PR.
ionitron-bot commentedon Dec 21, 2025
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.