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

draggable example account for the start position #170

Closed
wants to merge 1 commit into from
Closed

draggable example account for the start position #170

wants to merge 1 commit into from

Conversation

cybersoulK
Copy link
Contributor

Draggable example account for the start position

@cybersoulK
Update draggable.rs
9e74c66 
Draggable example account for the start position
@cybersoulK cybersoulK requested a review from LPGhatguy as a code owner June 19, 2024 02:23
@LPGhatguy
Copy link
Member

I tested both the current main branch and this PR. The draggable example currently does work and the change from this PR is unexpected.

Current Main Branch

draggable_tduMSDMt0A.mp4

This PR

draggable_Eda2OkLMB4.mp4

@LPGhatguy LPGhatguy closed this Jul 13, 2024
@cybersoulK
Copy link
Contributor Author

cybersoulK commented Jul 13, 2024 •
edited
Loading

it has been quite a while since then,
but i am pretty sure that if the element is located somewhere else other than the Left Top corner (you can try padding), then the current example does not work well, like video N.1

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

@LPGhatguy LPGhatguy Awaiting requested review from LPGhatguy LPGhatguy is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cybersoulK @LPGhatguy