plotly.js icon indicating copy to clipboard operation
plotly.js copied to clipboard

provide option to display totals on top of each bar

Open wilhelmhb opened this issue 4 years ago • 13 comments
trafficstars

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

wilhelmhb avatar Jan 06 '21 08:01 wilhelmhb

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.

wilhelmhb avatar Jan 06 '21 08:01 wilhelmhb

Any update on this ?

wilhelmhb avatar Mar 09 '21 08:03 wilhelmhb

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 :)

nicolaskruchten avatar Mar 09 '21 14:03 nicolaskruchten

I am interested in using this feature. Any update?

mysheepb avatar Oct 12 '21 05:10 mysheepb

@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 avatar Mar 15 '22 13:03 vovavili

@vovavili could you please fetch upstream/master and merge it into this branch?

archmoj avatar Mar 23 '22 15:03 archmoj

I am interested in using this feature. Any update?

hjc0826 avatar May 23 '22 10:05 hjc0826

@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

wilhelmhb avatar Jul 13 '22 16:07 wilhelmhb

Hi there is any update?

hadarby avatar Jul 31 '22 15:07 hadarby

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!

archmoj avatar Nov 07 '22 19:11 archmoj

October 2023, I could really use this feature. Thank you.

Pampelmops avatar Oct 12 '23 10:10 Pampelmops

@archmoj @alexcjohnson How is the review going? It would be really helpful to display totals for bar charts.

futurejohn avatar Dec 20 '23 19:12 futurejohn