[OP-152] Rename btn-delete to btn-destructive

[Jeremy-Walton]

Why?

From a semantic perspective, delete doesn't match other buttons. The original reason for the naming was to convey more intention when using it. destructive is just as semantic, but allows for broader use cases like closing an account (that may be recoverable)

What Changed

• Add btn-destructive class
• Update documentation terminology
• Retail btn-delete so as to not cause breaking changes

Sanity Check

• [ ] Have you updated any usage of changed tokens?
• Have you updated the docs with any component changes?
• [ ] Have you updated the dependency graph with any component changes?
• Have you run linters?
• Have you run prettier?
• Have you tried building the css?
• Have you tried building storybook?
• [ ] Do you need to update the package version?

Screenshots

[linear]

OP-152 Is there a reason for the discrepancy between the `danger` color and the button being called `delete`?

[scriswell]

@Jeremy-Walton I'm glad you made this backwards compatible with the older name.