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

Draggable row with sliders drag the row along with the slider #54

Open
alextd opened this issue Feb 3, 2023 · 0 comments
Open

Draggable row with sliders drag the row along with the slider #54

alextd opened this issue Feb 3, 2023 · 0 comments

Comments

@alextd
Copy link

alextd commented Feb 3, 2023

video via bug report: https://imgur.com/R4MRtLh

So the problem is this transpiler, TL;DR remove it, why does it even exist?: https://github.com/alextd/kNumbers/blob/642fbaf518defc8c3b09d3d1f4a535d51bc50a52/Numbers/Numbers.cs#L192

It changes the vanilla code for reorderable dragging from event-based to input-based (for some reason)

When the slider in the reorderable row captures and uses the event, the reorderable code that follows should not use the event and should not drag.

But after the transpiler, it's using input instead of events, it detects that the mouse is down anyway and drags the thing.

So it oughta not do that? It ought not transpile that method at all? What is it even doing that for? This is old old code though so gee, who knows why it ever existed.

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

No branches or pull requests

1 participant