Add empty state to chart widget
Description
[!CAUTION] DON'T MERGE THIS YET!
@danharrin as discussed in https://github.com/filamentphp/filament/discussions/13311 I have added a feature to display an empty state on chart widgets. Before this is merged, I'd like to ask a couple of things as I am not entirely familiar with everything in Filament:
- I added a string and translations for
enandde. Is that enough? How are all the other translations handled for so many languages? - I didn’t use screenshots or links in the docs as I didn’t know how to do them.
- The implementation is very much based on the work done on tables. But as we aren't defining these things by calling methods on an object but rather by defining a class that extends ChartWidget, I removed all the setters from it. But I guess that also means closures don't really make sense for the attributes or do they? I want to make sure this is in the spirit of Filament so any guidance would be much appreciated.
- Is everything else as you'd expect? Anything I missed?
Let me know and I'll get the rest sorted. Otherwise, it's ready to be tested.
Visual changes
Functional changes
- [x] Code style has been fixed by running the
composer cscommand. - [x] Changes have been tested to not break existing functionality.
- [x] Documentation is up-to-date.
@zepfietje that would make a ton of sense. Let me know when that’s done or if you need any help just instruct me.
Great! It's at the top of my list, so will get to it as soon as possible.
I'll mark this PR as draft until we can move forward. 👍
Started working on https://github.com/filamentphp/filament/pull/13642.
Going to close this since there was no movement on #13642. Next step would be putting in a v4 PR for an empty state Blade component, and then opening another PR to add that to the chart widget.