Mehni / kNumbers

Quick comparison of colonist stats
MIT License
34 stars 28 forks source link

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

Open alextd opened 1 year ago

alextd commented 1 year ago

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.