Skip to content

Add cell context menu entry to delete column #4918

@seancolsen

Description

@seancolsen

Current behavior

  • We have this row context menu entry to delete rows

    Image
  • But for columns, there is no way to delete a column from the context menu

    Image

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

No one assigned

    Labels

    good first issueEverything in "Help wanted", PLUS being relatively easy and straightforward to implement.help wantedCommunity contributors can implement thisreadyReady for implementationtype: enhancementwork: frontendRelated to frontend code in the mathesar_ui directory

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions