-
Notifications
You must be signed in to change notification settings - Fork 13.4k
feat(select-modal): add cancelText prop #30282
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: feature-8.8
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Thank you for submitting the PR! Please create an issue for this. We'd like to understand the use case for this feature before we expose this API. Thanks! |
| component: 'ion-select-modal', | ||
| componentProps: { | ||
| header: interfaceOptions.header, | ||
| closeText: interfaceOptions.closeText, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cancelText: this.cancelText,
I consider better to use the existing property cancelText, instead of creating a new one, so we avoid having two properties for the same thing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with you @luisbytes. Thanks for letting me know.
I'll change it and push the changes today.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done @luisbytes 😄
|
Is there any date when this feature might get released? |
|
We appreciate your interest in this feature! While the team does plan to address it at some point, there's no set timeframe for its release. We recommend subscribing to the issue for any future updates. |
Add a new property to customize the close button in the select when the interface is Modal.
14c7d9c to
5225f7b
Compare
|
@thetaPC is attempting to deploy a commit to the Ionic Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The difference is the cancel button text. Since we are passing the cancel button text from select, it now uses the default value that select sets for cancel button. I believe this is actually a fix because it lines up with the other interfaces.
Issue number: resolves #30295
What is the current behavior?
The cancel button within
ion-select-modalcannot be changed.What is the new behavior?
cancelText, toion-select-modalto customize the cancel button text.ion-selectto pass thecancelTextvalue toion-select-modal.Does this introduce a breaking change?
Preview