plotly.js
plotly.js copied to clipboard
provide option to display totals on top of each bar
This PR aims at allowing users to display the total of each bar on top of it (bottom of it for negative values).
TODO:
- [X] Need description: see #85
- [X] Base implementation
- [X] Lint
- [ ] Add tests
- [ ] Add documentation
I haven't yet been able to run the tests, probably a browser/configuration issue. Thus I haven't started on the jasmine tests. I can provide a few test cases if needed. I don't really see where the documentation is written, but if someone points me in the right direction, I'll be glad to add a few examples.
Any update on this ?
Thanks for this PR! Our team is currently working hard on finishing up the 2.0 release of Plotly.js which explains our radio-silence for the past while, and I apologize for not communicating more clearly about it. This PR should probably land in 2.1, which we expect to come out in April. I'm sorry for the long timelines here :)
I am interested in using this feature. Any update?
@nicolaskruchten Hello! I am interested in using this feature. Now that we're reaching April 2022 and plotly 2.11, are there any updates on this PR?
@vovavili could you please fetch upstream/master and merge it into this branch?
I am interested in using this feature. Any update?
@nicolaskruchten @archmoj I've just merged upstream/master in this branch
I removed a typo, and added an example in test/image/mocks/bar_stackrelative_negative_total.json, available when you run npm start, then lookup the filename in the mocks search field.
I personally have a weird result on my machine. However, as I have the same result when using any text with textposition : outside, I'm assuming this is an issue with plotly.js, which should be a separate matter.
I'm still unable to run the jasmine tests on my machine.
Hoping this can still make it to 2.13
Hi there is any update?
Thanks @wilhelmhb for the great PR.
My apologies for the late review.
Kindly find my comments regarding the new attributes.
Before adding new tests, I also suggest that you possibly pull upstream/master and merge it to your branch.
@alexcjohnson it would be great if you also provide a review. Thank you!
October 2023, I could really use this feature. Thank you.
@archmoj @alexcjohnson How is the review going? It would be really helpful to display totals for bar charts.