Skip to content

refactor(ui5-toast): replace show method with open property#8855

Merged
ilhan007 merged 3 commits intomainfrom
toast-open
Apr 26, 2024
Merged

refactor(ui5-toast): replace show method with open property#8855
ilhan007 merged 3 commits intomainfrom
toast-open

Conversation

@MapTo0
Copy link
Copy Markdown
Contributor

@MapTo0 MapTo0 commented Apr 26, 2024

Users now can open ui5-toast using the open property. A new event after-close is introduced.
It may be used to sync app state with the open state of the ui5-toast.

BREAKING CHANGE: The Toast#show method has been replaced by open property. If you previously used toast.show() to show the toast, you must now se toast.open=true.

Related to: #8461

Users now can open `ui5-toast` using the open property.
A new event `after-close` is introduced.
It may be used to sync app state with the `open` state of the `ui5-toast`.

BREAKING CHANGE: removed `show()`, set `open` property to true instead.

Related to: #8461
@MapTo0 MapTo0 requested review from a team, ilhan007 and nnaydenow April 26, 2024 08:02
/**
* Fired after the component is auto closed.
* @public
*/
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

add @since 2.0.0 tag. same for open property

@ilhan007 ilhan007 merged commit 372d27d into main Apr 26, 2024
@ilhan007 ilhan007 deleted the toast-open branch April 26, 2024 15:03
nnaydenow pushed a commit that referenced this pull request Apr 29, 2024
Users now can open `ui5-toast` using the `open` property. A new event `after-close` is introduced.
It may be used to sync app state with the `open` state of the `ui5-toast`.

BREAKING CHANGE: The Toast#show method has been replaced by  `open` property. If you previously used  `toast.show()` to show the toast, you must now se `toast.open=true`.

Related to: #8461
@ilhan007 ilhan007 mentioned this pull request May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants