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.

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

Pass on pull handler #255

Merged
merged 1 commit into from
May 22, 2023
Merged

Pass on pull handler #255

merged 1 commit into from
May 22, 2023

Conversation

tinchodias
Copy link
Collaborator

  • Pull handler:
    • remove relocateX and Y (was too confusing)
    • rename targetElement: to target:
  • BlPullEvent:
    • add class comment
    • full change in API: names were confusing, and not used. Now the event knows current position and initial position and seems to be fine.

Why? check the following users of this change:

Pull handler:
- remove relocateX and Y (was too confusing)
- rename targetElement: to target:
BlPullEvent:
- add class comment
- full change in API: names were confusing, and not used. Now the event knows current position and initial position and seems to be fine.
@plantec plantec merged commit 3287844 into dev-1.0 May 22, 2023
@plantec
Copy link
Collaborator

plantec commented May 22, 2023

great!
I wanted to do the same.
thanks

@tinchodias tinchodias deleted the scrollbars branch May 23, 2023 00:34
@tinchodias
Copy link
Collaborator Author

:)
It wasn't easy to me to realize how relocateX was used by Toplo grips, so I was keeping it. But at some point I found that it wasn't worth having that since subscribing directly to drag events was fine.

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