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

Implement drag stream #65

Closed
appsforartists opened this issue Sep 15, 2016 · 4 comments
Closed

Implement drag stream #65

appsforartists opened this issue Sep 15, 2016 · 4 comments
Assignees
@appsforartists
Labels
Feature request flow: In review
Milestone
Draggable MVP

Comments

@appsforartists
Copy link
Member

appsforartists commented Sep 15, 2016 •
edited
Loading

Take a pointersStream (#64) as input and transform it into a stream of deltas. Every pointerdown/pointerup should move the starting point to the center of the current pointers. Every pointermove should dispatch a delta that represents the difference between the active pointers' center and the starting point.

Open questions:

  • What should the deltas be relative to (the last event or the starting event)?
    • Last event is much simpler to model - is it accurate?
  • How do you incorporate a target's current position (to make sure it moves relative to its last-known-position rather than always from {0,0})?
    • This should be the responsibility of a performer, not of the stream. The stream just represent changes in pointer location over time.
@appsforartists appsforartists added this to the Direct Manipulation Gesture Streams milestone Sep 15, 2016
@appsforartists appsforartists changed the title Drag stream Sep 15, 2016
@appsforartists appsforartists changed the title drag stream Sep 15, 2016
@appsforartists appsforartists self-assigned this Sep 16, 2016
@appsforartists
Copy link
Member Author

Pen
@appsforartists
Copy link
Member Author

💻 I'm working on a diff at http://codereview.cc/D1692
@jverkoey jverkoey removed this from the Direct Manipulation Gesture Streams milestone Dec 28, 2016
@jverkoey jverkoey added this to appsforartists (Brenton) in Ownership Mar 29, 2017
@appsforartists appsforartists mentioned this issue Apr 21, 2017
Closed
@appsforartists
Copy link
Member Author

The iOS and Android implementations dispatch values relative to pointerdown and add them to a view with translationAddedTo(). To maintain conceptually parity across the platforms, I'll likely start there too.
@appsforartists appsforartists added this to the Draggable MVP milestone Apr 21, 2017
@appsforartists
Copy link
Member Author

🎊 My diff is ready for review at http://codereview.cc/D3085
@appsforartists appsforartists removed this from appsforartists (Brenton) in Ownership Apr 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request flow: In review
2 participants
@jverkoey @appsforartists