-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Patch fixes: Eliminate plot_title for Replaced mode stacked bar charts #157
Conversation
…. We display the relevant information adjacent to the bar labels.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, we no longer need quality text in the title because it is in the bar label
@iantei it would help if you would describe the problem rather than your solution. What is the extra information? Why is it bad? That is not clear from your description, which increases the effort required to review and approve |
@iantei quick note: it would be helpful to have a more detailed commit message that links to the related discussion. Otherwise PR looks good. Thanks for keeping it clean! |
This PR incorporates the following changes: