Skip to content
This repository has been archived by the owner on Jun 22, 2022. It is now read-only.

[Feature request] Toast notifications need an overhaul #98

Open
nvms opened this issue Feb 23, 2019 · 0 comments
Open

[Feature request] Toast notifications need an overhaul #98

nvms opened this issue Feb 23, 2019 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@nvms
Copy link
Owner

nvms commented Feb 23, 2019

What problem does this feature solve?

the current implementation is.. underwhelming.

Ideas:

  • should look more similar to the Notification component
  • the first argument to the VaToast method should be a string and it should be the only required argument, so that they can be instantiated as such:
this.VaToast('this is my toast message')
  • should accept an optional config object where additional properties can be defined, such as:
    • description
    • duration
    • type
    • placement
@nvms nvms added the enhancement New feature or request label Feb 23, 2019
@nvms nvms self-assigned this Feb 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant