Skip to content
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

refactor(item): remove deprecated apis #29102

Merged
merged 29 commits into from
Mar 7, 2024
Merged

refactor(item): remove deprecated apis #29102

merged 29 commits into from
Mar 7, 2024

Conversation

liamdebeasi
Copy link
Contributor

@liamdebeasi liamdebeasi commented Feb 29, 2024

Issue number: Internal


What is the current behavior?

Item the following deprecated APIs:

Props

  1. fill
  2. shape
  3. counter
  4. counterFomatter

Slots

  1. helper
  2. error

What is the new behavior?

  • These properties and slots (including implementations) have been removed.

Does this introduce a breaking change?

  • Yes
  • No

Other information

BREAKING CHANGE:

  • The helper slot has been removed. Developers should use the helperText property on ion-input and ion-textarea.
  • The error slot has been removed. Developers should use the errorText property on ion-input and ion-textarea.
  • Counter functionality has been removed including the counter and counterFormatter properties. Developers should use the properties of the same name on ion-input and ion-textarea.
  • The fill property has been removed. Developers should use the property of the same name on ion-input, ion-select, and ion-textarea.
  • The shape property has been removed. Developers should use the property of the same name on ion-input, ion-select, and ion-textarea.

@github-actions github-actions bot added package: core @ionic/core package package: angular @ionic/angular package package: vue @ionic/vue package labels Feb 29, 2024
@liamdebeasi liamdebeasi changed the title Fw 5289 refactor(item): remove deprecated apis Mar 6, 2024
@liamdebeasi liamdebeasi marked this pull request as ready for review March 6, 2024 11:11
core/src/components/item/item.md.scss Outdated Show resolved Hide resolved
core/src/components/item/item.ios.scss Show resolved Hide resolved
core/src/components/item/item.md.scss Outdated Show resolved Hide resolved
core/src/components/item/item.scss Outdated Show resolved Hide resolved
core/src/components/item/item.scss Show resolved Hide resolved
core/src/components/item/test/basic/item.e2e.ts Outdated Show resolved Hide resolved
core/src/components/textarea/textarea.scss Show resolved Hide resolved
liamdebeasi and others added 3 commits March 6, 2024 15:50
Co-authored-by: Amanda Johnston <90629384+amandaejohnston@users.noreply.github.com>
core/src/components/item/item.md.scss Outdated Show resolved Hide resolved
core/src/components/item/item.md.scss Outdated Show resolved Hide resolved
Copy link
Contributor

@mapsandapps mapsandapps left a comment

Choose a reason for hiding this comment

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

I think I found two other things that can be removed:

@liamdebeasi
Copy link
Contributor Author

Bottom styles and variables removed in d482246 and f98e95d.

@liamdebeasi liamdebeasi merged commit 743f517 into feature-8.0 Mar 7, 2024
44 checks passed
@liamdebeasi liamdebeasi deleted the FW-5289 branch March 7, 2024 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: angular @ionic/angular package package: core @ionic/core package package: vue @ionic/vue package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants