[UI] Support deleting tags from the run view UI

[harupy]

What changes are proposed in this pull request?

Support deleting tags from the run view UI

How is this patch tested?

Manually tested

Release Notes

Is this a user-facing change?

• No. You can skip the rest of this section.
• Yes. Give a description of this change to be included in the release notes for MLflow users.

Run tags improvements. Adds the ability to delete tags from the run view UI.

What component(s) does this PR affect?

• UI
• CLI
• API
• REST-API
• Examples
• Docs
• Tracking
• Projects
• Artifacts
• Models
• Scoring
• Serving
• R
• Java
• Python

How should the PR be classified in the release notes? Choose one:

• rn/breaking-change - The PR will be mentioned in the "Breaking Changes" section
• rn/none - No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" section
• rn/feature - A new user-facing feature worth mentioning in the release notes
• rn/bug-fix - A user-facing bug fix worth mentioning in the release notes
• rn/documentation - A user-facing documentation change worth mentioning in the release notes

[smurching]

Thanks @harupy for the PR! cc @gioa any thoughts on the UX here? Maybe we should prompt the user for confirmation before deleting the tag? Also, seems a little risky to have "Delete" so close to "Save"

[harupy]

Agree with @smurching. We should at least prompt the user because this operation is irreversible.

[gioa]

@harupy Thank you for working on this! +1 on adding a delete confirmation. Also, we can add the delete icon near the edit to separate the flow for these two actions. See an InVision prototype here: https://databricks.invisionapp.com/share/36NY54RMWPJ#/319871033_Edit-Inline

[smurching]

Thanks @gioa for the design! @harupy you can likely implement the confirmation flow using antd's popconfirm component: https://ant.design/components/popconfirm/

[harupy]

@gioa @smurching Thanks for the suggestions! I just updated the code. Now it looks like:

[gioa]

LGTM!

[smurching]

LGTM as well, will merge once tests pass - thanks @harupy!

[harupy]

test_model_save_load in tests/keras/test_keras_model_export.py failed.
https://travis-ci.org/mlflow/mlflow/jobs/598638360#L2340

[harupy]

oh this bug is mentioned in #1927

[harupy]

@smurching @gioa
The Travis build passed this time.

[harupy]

@smurching Thanks!