search

GrammarViz2 / grammarviz2_src

GrammarViz 2.0 public release:
http://grammarviz2.github.io/grammarviz2_site
GNU General Public License v2.0
120 stars 39 forks source link

Humble request - please enable panning. #12

Closed mmehrle closed 9 years ago

mmehrle commented 10 years ago

I looked through the JFree library and realize there may be some effort involved but it seems that panning is well documented and shouldn't be hard to implement. I'm loading years of time series data - 10,000 data points and more - up to 50,000. Grammarviz is handling it extremely well actually, no complaints. But being able to pan around instead of zooming in and then zooming all the way out would accelerate things quite a bit for me. Thanks in advance and thank you for your efforts - I LOVE this tool.

seninp commented 10 years ago

I had a quick look at the web code samples, it is not immediately clear how to enable this feature, but seems like the few recent discussions address that

mmehrle commented 10 years ago

Ah - that's great! I'll give it a try - assuming I figure out how to apply those patches ;-)

seninp commented 10 years ago

Well, if you'd find a solution - please make a pull request - I guess that every other user will thank you a lot. I am reopening the issue meanwhile.

mmehrle commented 10 years ago

Okay, this is what I found when I poked around last week:

http://svn.codehaus.org/gumtree/gumtree/tags/REL-1.5.0/org.jfree.jfreechart/src/org/jfree/panning/PanningDemo.java

I don't have any hands-on experience with JFree but it seems that it's relatively simple. Again, not wanting to distract you from more pressing deliveries.

seninp commented 10 years ago

Right, I saw that - and this is confusing because I checked within JFree jars and found that these classes do not exist. I'll get to this issue probably next week. Thanks again for bringing this issue up - I had the same difficulty as you - zooming in and out and going to next location, do the same etc. Having panning going to be a permanent fix to that nonsense. By the way, what do you use this code for? If you can, please PM me, it is always exciting to learn new things. Thanks!

mmehrle commented 10 years ago

I did the same thing actually - funny. The idea here is that it's a reference implementation - it's not in any of the builds I looked at. So it would have to be replicated - it's just a recipe on how to do it.