villebro commented on PR #36239: URL: https://github.com/apache/superset/pull/36239#issuecomment-3598987870
Great work! I especially LOVE the way reordering now visually shows how the columns/metrics shuffle around (compared to how the controls work). However, while playing around with this, one critique I have is that while it's great that you can reorder controls/metrics as you wish into folders, we currently don't distinguish between controls and metrics like we do in the dataset panel: <img width="618" height="872" alt="image" src="https://github.com/user-attachments/assets/e89b3711-21fa-4b88-a671-8af11af810d8"; /> Here in the **test** folder we're mixing both metrics and columns, which can be very confusing, especially if you have lots of them. I don't have a great proposal for fixing this, but a few things come to mind: 1. Maintaining separate columns and metrics within the folder view, like we currently do for un-foldered columns/metrics. This would clarify which is which, but as a downside, it'll eat up more real estate. 2. Have a folder selector above the metrics/columns section in the dataset panel, which would then act as a filter for the columns/metrics sections below. This would maintain the current UX, but as a downside, you wouldn't have the same freedom of browsing or reorganizing columns/metrics. 3. Change the labels of the columns/metrics to clearly show what type of elements they are. This would disambiguate them, but would maybe add more overhead in terms of having the new icon/indicator on each element. Also, when folders are defined, I wonder if we should just dump un-foldered columns/metrics under an "Undefined" folder which would be the last top level folder in the tree. Unfortunately I don't have any great solutions here, just bad options.. @kasiazjc do you have any thoughts on how to harmonize the folder view with the current UX? -- 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]
