Skip to content

Fix #87 support d3 v4, and add horizontal zoom (fixes #57)#93

Open
fidlr wants to merge 5 commits intojiahuang:masterfrom
fidlr:master
Open

Fix #87 support d3 v4, and add horizontal zoom (fixes #57)#93
fidlr wants to merge 5 commits intojiahuang:masterfrom
fidlr:master

Conversation

@fidlr
Copy link

@fidlr fidlr commented Mar 13, 2017

No description provided.

fidlr added 2 commits March 13, 2017 22:03
Added horizontal zooming with mouse wheel (closes jiahuang#57). Code cleanup.
Fixed newely introduced bugs: (1) background occluding data rects, (2) support for xaxis orientation top.

Added grouping of the data rects in the svg (viewport).
@fidlr fidlr changed the title Fix #87 support d3 v4 Fix #87 support d3 v4, and add horizontal zoom (fixes #57) Mar 19, 2017
@fidlr fidlr mentioned this pull request Mar 19, 2017
@kwe7
Copy link

kwe7 commented Mar 20, 2017

You have saved my week fidlr! This is fantastic. Thank you.

@fidlr
Copy link
Author

fidlr commented Mar 21, 2017

Happy to help @kwe7

@kwe7
Copy link

kwe7 commented Mar 23, 2017

Hi fidlr, do you know how to apply clipping to your zoom feature? The zoom works perfectly, but the timeline will expand to block my view of the y-axis values.

Also, I think this might be more of a me problem, but upon following your v4 updates, my time axis shows correct minutes but not correct hours. d.starting_time and d.ending_time are still correct. Do you have any insight as to why only my time axis was affected in this way?

Thanks!

@fidlr
Copy link
Author

fidlr commented Mar 26, 2017

I have the same issue with the labels, but it was like that in the original code with scrolling so I left it as-is for now. I might get back to it later if it becomes an issue. If you post a fix, I'd definitely want it.
As for the date, I might have had the same problem, not sure. Make sure you call "new Date(d)", instantiating a new Date object.

@kwe7
Copy link

kwe7 commented Mar 28, 2017

I think that I have successfully added clipping for the zoom. How do I add a commit so everyone can see the changes?

@fidlr
Copy link
Author

fidlr commented Mar 30, 2017

You can either fork and submit a pull request, or attach a patch, and I'd be happy to merge it to this fork.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants