Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Aggregated pull requests (with pull request 62) #63

Closed
wants to merge 28 commits into from
Closed

Aggregated pull requests (with pull request 62) #63

wants to merge 28 commits into from

Conversation

arlm
Copy link
Contributor

@arlm arlm commented Feb 7, 2013

It aggregates:

pkendall64 and others added 28 commits October 23, 2012 10:30
@pkendall64
Allow forceY on lines1 and lines2
623a1d0
@kapexx
Fixed rotateLabels option for multiBarCharts
4681078 
Replaced 'text-transform' with correct attribute 'text-anchor' to set
rotation center of labels.
@tve
added lineTimeSeriesChart with example
ebb1bc5
@tve
added initial support for zoom and pan to dynamic line time series
aa592e5
@jonaldomo
adding d3.event to event listeners
75e9410 
Adding d3.event to event listeners to maintain consistency between models.  In the historical bar chart they are included, but not the line which uses the scatters events.
@mookie5dc
Added donutRatio public variable
efbec8a 
Added donutRatio public variable to pie model.  Allows customization of the inner radius of the donut pie chart.
@mookie5dc
rebind donutRatio for pieChart
99ce4ae
@ggriffiniii
Adjust axis labels on update rather than enter.
c6c89b7 
This allows an axis to change orientation and have the tick label respond accordingly.
@jonaldomo
exposed public variable brushExtent
6f54973 
the variable brushExtent is documented as a public variable but is not currently exposed.  this exposes it allowing the user to control which portion of the crossfilter is selected at initiation.
@jonaldomo
updating spacing
fc4b9c7
@awakia
Fix bug of updateFisheye when you filtered out some data
28572f7
@arlm
* Fix on Lee Gillentine's commit 2ac200e
ff79dd2 
Cannot read property 'values' of undefined nv.d3.js:8498
Cannot read property 'values' of undefined nv.d3.js:8511
@arlm
Merge branch 'fix' into fix-test
d1e307a
@arlm
* Fix on twolf's commit 8e0160a
e26d3af 
Problem parsing d="MZ" error
@arlm
Merge branch 'Branch_8e0160ad5f15f3c9d28a38c9b2c7ed1fc48e9f8c' into fix
7566162 
Conflicts:
	nv.d3.js
	nv.d3.min.js
@arlm
Merge branch 'master' of github.com:novus/nvd3
060836a
@arlm
Merge branch 'master' into fix
139720f 
Conflicts:
	nv.d3.js
	nv.d3.min.js
@arlm
* Fix name clash on Fisheye components
fa94c57
@arlm
Merge remote-tracking branch 'remotes/wantedly/fix-update-fisheye-of-…
7f09fab 
…scatter-chart' into aggregated-pull-requests
@arlm
Merge commit 'c6c89b7e0723a3ba92522a3b2c98dc9d2d651536' into aggregat…
8cf95de 
…ed-pull-requests
@arlm
Merge commit 'efbec8ae3c448e29ba1d953ced825547a1886830' into aggregat…
39d5199 
…ed-pull-requests

Conflicts:
	src/models/pie.js
@arlm
Merge commit '99ce4ae6677983bfcb71b3f9175be692a97c95cb' into aggregat…
eeb90a5 
…ed-pull-requests
@arlm
Merge remote-tracking branch 'remotes/jonaldomo/master' into aggregat…
d6f567d 
…ed-pull-requests
@arlm
Merge commit 'ebb1bc5304096dc6399d82d89bd1da37b43cf295' into aggregat…
570866c 
…ed-pull-requests
@arlm
Merge remote-tracking branch 'remotes/tve/master' into aggregated-pul…
7144906 
…l-requests
@arlm
Merge commit '623a1d05481ef3a417afad0e07ddfb5ea10a4f2e' into aggregat…
ba4401a 
…ed-pull-requests
@arlm
Merge remote-tracking branch 'remotes/kapep/master' into aggregated-p…
789ab49 
…ull-requests
@arlm
* Small fix on dispatch.elementMouseover event
5f0e6cd
@bobmonteverde
Copy link
Contributor

Thank you so much, sorry I've been behind. How are things looking, I'll have to dig into the code a little, but if I hear things appear to be working as they should, I'll dig a little less deeper, and just do an overview of all the pull requests.

@arlm
Copy link
Contributor Author

arlm commented Feb 8, 2013

I did a throughout test on everything. This pull request includes the bug fixes I did on the other pull request. I did it separated also to make it easier for you to review the bug fixes. There were three outstanding issues on the master branch of this repository.

@bobmonteverde
Copy link
Contributor

OK, sorry, snowstorm whiped out my weekend. I'll try to get this in in the next day or two so I can get this repo back up to date.

@arlm
Copy link
Contributor Author

arlm commented Feb 13, 2013

No problem, I'm working on an issue that I have involving line charts, date/time X scales and redrawing data. Soon I'll have something to share about it.

@arlm
Copy link
Contributor Author

arlm commented Feb 13, 2013

trying to do my sample I encountered a bug that is not present on this repository master. If possible, help me find it.

It is reported on Bug report #67

@robinfhu
Copy link
Contributor

robinfhu commented Feb 9, 2015

Too many PR's in this, so closing.

@robinfhu robinfhu closed this Feb 9, 2015
liquidpele added a commit that referenced this pull request Feb 11, 2015
@liquidpele
Merge pull request #63 from jamesgarfield/ClearOnNoData
f63afff 
Clear chart on no data #2
@juhamust juhamust mentioned this pull request Feb 2, 2016
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@arlm @bobmonteverde @robinfhu @pkendall64 @kapexx @tve @jonaldomo @mookie5dc @ggriffiniii @awakia