appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

[Bug]: Dragging a List Widget Inside Another List Widget Crashes the App

Open ame-appsmith opened this issue 2 years ago • 3 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Description

Although currently, nested lists are not supported, if the user resizes a list, they can make it fit inside another list, which makes the app crash. Users should not be allowed to add a list widget inside another list widget unless nested lists will be supported.

Loom screen-recording: https://www.loom.com/share/88ff801b01154010996fb73dcb261f90

https://app.intercom.com/a/apps/y10e7138/inbox/inbox/conversation/164629100139777#part_id=initial-part-164629100139777-1365134472

Steps To Reproduce

  1. Add a List widget to the canvas.
  2. Add a second List widget to the canvas.
  3. Resize one of the List widgets, decreasing its size until it fits inside the item of the other List widget.
  4. Drag it inside the "parent" List widget and drop it once it's not highlighted with red anymore.
  5. Note that the app crashes and console errors get displayed.

Public Sample App

No response

Version

Cloud v1.7.7-SNAPSHOT

ame-appsmith avatar Jul 08 '22 14:07 ame-appsmith

Thanks for adding! :)

unknown1337 avatar Jul 08 '22 14:07 unknown1337

FYI: In this same manner, it is possible to nest a Table Widget inside a List widget. The app does not crash in this case, but this operation should probably not be allowed.

ame-appsmith avatar Jul 08 '22 14:07 ame-appsmith

@Tooluloope is it possible to fix this independently without solving it in list widget epic?

dilippitchika avatar Jul 11 '22 12:07 dilippitchika

Note: Fixed in List v2 POC.

ashit-rath avatar Oct 21 '22 03:10 ashit-rath