justinpark commented on PR #26699: URL: https://github.com/apache/superset/pull/26699#issuecomment-1933167329
> one comment about the header thing Sophie mentioned: when you try to hover over it it shows a red box (error), which gave me an idea. I think maybe it could be used in places where there is not enough space to place the item, for example if the row is full. I had a similar thought as well, but there are complexities when it comes to dropping an existing component into a space that's just enough. Specifically, it's challenging to programmatically differentiate this from the prohibited drop zones. (For instance, the highlight box should not turn red when item [1] is dropped into a zone without space, as it's possible to reorder among the same row items. However, it should turn red when item [4] is dropped into a zone without space.) |[ 1 ]|[ 2 ]|[ 3 ][no space drop zone]| -------- |[ 4 ]|[ ]| Therefore, I've decided to retain the post notification as it is currently displayed. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
