kasiazjc commented on PR #33002: URL: https://github.com/apache/superset/pull/33002#issuecomment-2944977472
> are we good to merge this one for now then? > > > Though from my perspective the current UX (as of this PR) is "ok for now". thanks @vedantprajapati and sorry for the long wait! I think the perfect case scenario would be to combine all actions in a dropdown as we actually have 4 actions @mistercrunch: - delete - certify - export - tag (which is not really visible) I would suggest using [this](https://ant.design/~demos/dropdown-demo-loading) component, the one with the caret down <img width="91" alt="image" src="https://github.com/user-attachments/assets/e3e28a0a-a302-4c13-8a26-645153984b0c"; /> Primary action would be export and all of the other ones would be in the dropdown menu (certify, tag, delete) and it would be displayed next to "deselect all" instead of all of the buttons. Thoughts? -- 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]
