justinpark opened a new pull request, #27558: URL: https://github.com/apache/superset/pull/27558
### SUMMARY The current explore drag-and-drop (DND) indicator only highlights the drop allow zone, and users can recognize the disallowed zone when they try to drop an item there. However, this UX can be challenging when the current dragging item is not droppable in the entire zone, as there is no highlighting to indicate this. https://github.com/apache/superset/assets/1392866/68d359bf-e8b7-44ff-bafb-e1b2055a0d48 This lack of feedback can lead users to believe that the DND functionality is not working, until they mouse over the zone. This commit aims to address this UX issue by implementing changes. Now, all droppable zones will be highlighted when dragging, and a different color scheme will indicate which zones are available and which are unavailable for dropping. https://github.com/apache/superset/assets/1392866/0f795241-9eeb-4a53-92c2-00a6a8d3d282 Additionally, the commit also updates the drag over feedback to align with the changes made to the dashboard's DND UX. This way, users can have a consistent and improved drag-and-drop experience, with clear usability feedback. ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF Before: https://github.com/apache/superset/assets/1392866/7b0a92b6-78f3-4aec-b679-023a048f90ab After: https://github.com/apache/superset/assets/1392866/6c796977-ae3d-478e-acf7-b4485e6eebe9 ### TESTING INSTRUCTIONS Create a "Table" chart Select RAW RECORDS and then drag the Metrics to check where is droppable zone ### ADDITIONAL INFORMATION <!--- Check any relevant boxes with "x" --> <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue --> - [ ] Has associated issue: - [ ] Required feature flags: - [x] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
