Paper scroller can't slide in both axis at the same time (using touchpad)

[Alexander Nestorov]

Using my mac's touchpad (or my magic trackpad) I can slide the paper scroller of my paper on one direction at a time. I can either scroll it on the X axis or on the Y axis, but I can't scroll on both axis at the same time.

I'm not sure if this can be reproduced with a mouse (most probably not), but it can be reproduced with any touchpad that supports multidirectional scrolling (2 axis at the same time).

Is this expected behaviour or is that a bug? Or maybe just missing feature?

[Alexander Nestorov]

I wasn't sure this could even be achieved in a web-browser, but it seems that it is possible.

Here is a demo: https://codesandbox.io/s/unruffled-butterfly-k1h9c?file=/src/index.js

What do you think @Roman? Worth implementing?

[Roman Bruckner]

Hi Alex,

to be honest I am not sure why this is not the default behaviour in browsers. The PaperScroller does not handle this at all. We let the browser do the user scrolling (just by having a container with overflow: scroll).

You can scroll manually on mousewheel as shown here.

            paperScroller.lock();

            paperScroller.el.addEventListener('mousewheel', function(evt) {

                const dx = evt.deltaX;
                const dy = evt.deltaY;

                paperScroller.el.scrollLeft += dx;
                paperScroller.el.scrollTop += dy;

            }, false);

Best,

Roman

--

---
You received this message because you are subscribed to the Google Groups "JointJS" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jointjs+u...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jointjs/d22e1612-80d7-4a10-94d2-3d74826685afn%40googlegroups.com.

--

Roman Bruckner, CTO

client IO

http://client.io

http://jointjs.com

+420 776 504 406

[Alexander Nestorov]

Hi Roman!

Certainly it feels like this should be the default behaviour in web browsers, but unfortunately that's not the

case in any of the web browsers I tried.

A related feature would be inertia-drag. Are you open to patches? I'm interested in both things and I

might as well write the code myself if JointJS won't support that. I'm open to submitting my code if 

you feel you can merge it.

Regards

You received this message because you are subscribed to a topic in the Google Groups "JointJS" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/jointjs/RCEtwGeQJus/unsubscribe.
To unsubscribe from this group and all its topics, send an email to jointjs+u...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/jointjs/CAB8Z7Xe1h0ToW8aiJ70x0xQsrNv3jLh06_%2BXa_Nvm0FYTxLSmQ%40mail.gmail.com.

[Roman Bruckner]

Hi Alex,

we're definitely open to accepting pull requests with new features.

JointJS - just open a PR on GitHub

Rappid - please PM me

Thank you,

Roman

To view this discussion on the web visit https://groups.google.com/d/msgid/jointjs/E63180CF-A556-46E0-A3C4-3328D4956622%40gmail.com.

[Alexander Nestorov]

Great!

Were would you like to have the inertia-drag and the bidirectional scrolling features? JointJS or Rappid?

To view this discussion on the web visit https://groups.google.com/d/msgid/jointjs/CAB8Z7Xe_mqoT0MrspKSdqCFO3eURBNgQg%3DDK5am7vUq5_hQMpw%40mail.gmail.com.

[Roman Bruckner]

I think it would make sense to implement this as part of Rappid's PaperScroller (JointJS Paper does not scroll). If a lower-level feature/change related to the Paper is required, this could be added to JointJS.

Best,

Roman

To view this discussion on the web visit https://groups.google.com/d/msgid/jointjs/2C7B4843-4A7A-4A33-BD4F-30A6CCB82DB0%40gmail.com.

[Alexander Nestorov]

Sounds good! 

I'll start working on this shortly!

Regards

To view this discussion on the web visit https://groups.google.com/d/msgid/jointjs/CAB8Z7XfaW4ApSq00%3D%2BiL7%3D%3DbQha8Z4Ph21eYe4C96P6Cfr9_Vw%40mail.gmail.com.

[Alexander Nestorov]

Patch has been submitted :) 

[Alexander Nestorov]

Hi Roman!

I saw the Rappid 3.5.0 release (and the JointJS 3.5.x releases)! Kudos for the great work!

I spent yesterday evening upgrading to the new version of both libs and I haven't encountered any regressions so far, which is great!

I also made a full diff of the source code just to check what/how the code changed. While I do see some bug fixes regarding

the mousewheel, I couldn't find neither the "scroll-both-axis-at-the-same-time" patch nor the "pitch-to-zoom" patch that I sent

you some time ago and I just wanted to confirm if you decided to exclude them from this release and if you plan to merge them in future releases.

Regards!

[Roman Bruckner]

Hi Alex,

Thank you for the feedback on Rappid v3.5.

The "scroll-both-axis-at-the-same-time" patch and the "pinch-to-zoom" features have been excluded from this release and we do have plans to add them in the next release. I have your codepen, just have to turn this into a feature and test it in all browsers.

Best,

Roman

To view this discussion on the web visit https://groups.google.com/d/msgid/jointjs/49095a75-70a7-4384-9ff3-2097e1031510n%40googlegroups.com.

[Alexander Nestorov]

Hi Roman!

I just wanted to check if you had the chance to merge this feature into jointjs+ (kudos for the name change!)

Regards!

To view this discussion on the web visit https://groups.google.com/d/msgid/jointjs/CAB8Z7Xd1mcWTtw5Sn2aj6GjdDy%2BqEEqLDJm04XJrAbemf-JnhQ%40mail.gmail.com.