search

RyanLua / Satchel

A modern alternative to Roblox's default backpack. Satchel aims to be more customizable and easier to use than the default backpack while still having a "vanilla" feel.
http://satchel.ryanluu.dev
Mozilla Public License 2.0
34 stars 4 forks source link

Remove usage of the Draggable property #9

Open VoiditeDev opened 1 year ago

VoiditeDev commented 1 year ago

Is your feature request related to a problem?

The Draggable property has been deprecated for years and shouldn't be really used, yet the code still uses it, which gives me a clue that it won't work in roblox anytime in the future

Describe the solution you'd like

It would've been better if it uses this module due to the fact that the Draggable property is deprecated

Additional context

You can check this topic out by a roblox staff

https://devforum.roblox.com/t/draggable-property-is-hidden-on-gui-objects/107689/5?u=vscplays

Code of Conduct

RyanLua commented 1 year ago

While Satchel does use the deprecated .Draggable internally, just like type safety, changing the dragging behavior to another which isn’t deprecated that high in our priority list. The current dragging system is fine now and poses no problems, rewriting or adding code to Satchel can add potentially add more bugs with little benefits.

If any issues with using the deprecated property do arise, I will definitely look to rewrite how dragging works.

https://devforum.roblox.com/t/2451549/40

RyanLua commented 1 year ago

Roblox just released plans for DragDetectors for UI, once this comes out I will definability plan on replacing Draggable with DragDetectors.

Screen Gui & Viewport Frame

  • ScreenGui/SurfaceGUI: we know you miss ‘draggable’ on 2DUI elements. We want to make it so that if you put a DragDetector below ScreenGui object like a button or text label, then you can drag it, complete with all the callbacks and API; you just won’t be able to bring it out of the plane.
  • ViewportFrame/WorldModels - we hope to make it so that DragDetectors work in this context just like they do in the main view.

Announcing DragDetectors! - Developer Forum

RyanLua commented 3 days ago

UI Drag Detectors have been released, though only for a few select experiences. I'll go and work on this (I'll assign myself when I do) when I have time and see if I can get it running.