add custom notification handling
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 18.3k
- Forks
- 2k
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 28
Description
Feature Request
In order for Plotly to better integrate with our design system, we would like to pass in a custom notifier callback.
Relevant code: notifier.js.
We cannot resolve this with CSS alone as there are interactions between notifications. For example, if we fired three consecutive notifications, they would stack. Using CSS alone, integrating with our existing notification framework is not feasible.
Thanks!
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with src/lib/notifier.js, the only relevant entry point named in the issue, and trace how notifications are created and stacked. Determine the callback interface needed for an external notification framework; done means callers can supply custom notification handling while consecutive notifications still work together.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100