Bar updates #12

Merged
merged 27 commits into from Nov 29, 2012

2 participants

@BRIMIL01

I have added the ability to add data labels to the stacked and regular bar graphs, similar to what was requested in issue 11. I have not accounted for the additional spacing necessary in the side bar and side stacked charts, do you have any suggestions for how to deal with that?

@donv donv was assigned Nov 10, 2012
@donv
Collaborator

Hi!

I am reviving the project and would like to merge your changes. Any chance you could rebase against the current master?

donv and others added some commits Nov 10, 2012
@donv donv Merge pull request #42 from dmaciejak/patch-1
Update lib/gruff/base.rb: set legend under the graph

Thank you for your contribution.
569d473
@donv donv * Added experimental charts to README. 6cd213f
@donv donv * Keep separate generated samples for mri and jruby to track differences 275c1c4
@donv donv * Keep separate examples of charts generated with java and mri to tra…
…ck differences.

* Uncovered bug in Java based DOT charts.  They are drawn transparently.
6426d24
@donv donv * Added dot and line chart examples to README. b09eede
@donv donv * Added more examples to readme. f3b3013
@donv donv * Added link to updated RubyDoc. 4ad0335
@donv donv * Updated link to source 104d361
@BRIMIL01 BRIMIL01 Added data value labels for bar graphs positioned above the bar ff2b00e
@BRIMIL01 BRIMIL01 Fixed issue when calculating significant, if marker_count is 0 there …
…is a divide by zero error
f64613d
@BRIMIL01 BRIMIL01 Added tests for bar value labels and fix for 0 marker count
Added commify to String to make numbers look pretty
Added label_format to allow for formatting of value labels
e7dd9b6
@BRIMIL01 BRIMIL01 Bar label values no longer affected by hide/show label marker boolean ab8359d
@BRIMIL01 BRIMIL01 Added side bar labels 4160d66
@BRIMIL01 BRIMIL01 Made it possible to use data labels for left bar labels in side bar g…
…raphs
9ecdfe5
@BRIMIL01 BRIMIL01 Added labels to side stacked bar chart 5688ace
@BRIMIL01 BRIMIL01 Removed explicit font settings from tests 5932d95
@BRIMIL01 BRIMIL01 Merge branch 'bar_updates' of github.com:BRIMIL01/gruff into bar_updates
Conflicts:
	lib/gruff/base.rb
	lib/gruff/side_stacked_bar.rb
	test/test_bar.rb
5259be2
@BRIMIL01

@donv Rebased off of master. Let me know if there is anything else I can help out with.

@donv donv merged commit cf1e090 into topfunky:master Nov 29, 2012

1 check passed

Details default The Travis build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment