Skip to content

refactor(toast): deprecate cssClass on ToastButton - #27959

Merged
liamdebeasi merged 4 commits into
feature-7.4from
toast-button-css-class
Aug 23, 2023
Merged

refactor(toast): deprecate cssClass on ToastButton#27959
liamdebeasi merged 4 commits into
feature-7.4from
toast-button-css-class

Conversation

@liamdebeasi

@liamdebeasiliamdebeasi commented Aug 9, 2023

Copy link
Copy Markdown
Contributor

Issue number: N/A


What is the current behavior?

As part of our work for the cancel CSS Shadow Part for toast buttons, we decided to deprecate the cssClass property on ion-toast. This feature stopped working once we migrated ion-toast to the Shadow DOM.

What is the new behavior?

  • Deprecated the cssClass property on IonToast.

Does this introduce a breaking change?

  • Yes
  • No

Other information

Do not merge until #27921 is merged.

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz CodeflowRun & review this pull request in StackBlitz Codeflow.

@github-actionsgithub-actionsBot added the package: core @ionic/core package label Aug 9, 2023
@liamdebeasiliamdebeasi changed the title chore: deprecate cssClass on ToastButtonchore(toast): deprecate cssClass on ToastButtonAug 9, 2023
@liamdebeasiliamdebeasi changed the title chore(toast): deprecate cssClass on ToastButtonrefactor(toast): deprecate cssClass on ToastButtonAug 9, 2023
@liamdebeasi
liamdebeasi marked this pull request as ready for review August 9, 2023 14:51
@liamdebeasi
liamdebeasi requested review from a team and sean-perkins and removed request for a teamAugust 9, 2023 14:51
@brandyscarney

Copy link
Copy Markdown
Member

Might want to point this at feature-7.4 now

@liamdebeasi
liamdebeasi changed the base branch from feature-7.3 to feature-7.4August 22, 2023 20:12
@liamdebeasi
liamdebeasi merged commit ae9f1ab into feature-7.4Aug 23, 2023
@liamdebeasi
liamdebeasi deleted the toast-button-css-class branch August 23, 2023 13:21
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core@ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@liamdebeasi@brandyscarney@sean-perkins