-
-
Notifications
You must be signed in to change notification settings - Fork 408
Open
Labels
good first issueEverything in "Help wanted", PLUS being relatively easy and straightforward to implement.Everything in "Help wanted", PLUS being relatively easy and straightforward to implement.help wantedCommunity contributors can implement thisCommunity contributors can implement thisreadyReady for implementationReady for implementationtype: enhancementwork: frontendRelated to frontend code in the mathesar_ui directoryRelated to frontend code in the mathesar_ui directory
Milestone
Description
Current behavior
-
We have this row context menu entry to delete rows
-
But for columns, there is no way to delete a column from the context menu
Desired behavior
- A "Delete Column" entry should be present at the bottom of the column context menu options. It should be red like the Delete Row entry.
Notes
-
With rows we have pretty good support for deleting multiple rows in bulk, so the row context menu allows deleting multiple rows at once.
-
With columns the story is different. In theory it would be possible to delete multiple columns at the API layer easily via RPC batching. But handling the outcome gets more complex. What if one column is deleted successfully while another column cannot be deleted? Handling this on the front end is not necessarily straightforward. So, to satisfy this issue, we should focus on deleting one column via the context menu. We can make a separate issue for bulk column deletion if necessary.
Metadata
Metadata
Assignees
Labels
good first issueEverything in "Help wanted", PLUS being relatively easy and straightforward to implement.Everything in "Help wanted", PLUS being relatively easy and straightforward to implement.help wantedCommunity contributors can implement thisCommunity contributors can implement thisreadyReady for implementationReady for implementationtype: enhancementwork: frontendRelated to frontend code in the mathesar_ui directoryRelated to frontend code in the mathesar_ui directory