feat(chip): add closeOnDelete prop

[jcfranco]

Related Issue: #6825

Summary

This prop will allow developers to configure whether a chip will close on Delete when focused.

[macandcheese]

Looks good! Couple q:

• Could you update the title from closeOnEscape?
• This works when both the entire chip is focused and when the close button itself is focused. Just confirming we want this to happen in both cases (I think we do)
• Should this also include backspace support? If so, do we need a different name (closeOnKeyboard... etc)

[jcfranco]

@macandcheese Excellent questions:

1. Fixed! My fingers keep typing Escape despite many edits for some reason. 🤣
2. I thought about ignoring Delete when the close button is focused, but it seems more consistent and straightforward to close regardless.
3. Solid point. Looking at our current Escape usage, aligning for consistency makes sense. I’m not sure what the prop should be named in this case. Open to suggestions here. 😅

[macandcheese]

I’m not sure what the prop should be named in this case. Open to suggestions here. 😅

Yeah - tough one to name. closeOnKeyboard with specific description could work. Or even leaving closeOnDelete with a description that includes "backspace and delete" keys could work.

Some keyboards (mainly ones marketed for Mac) have "delete" written on the backspace key anyway. For some laptops there is no "delete" standalone anyway. All that to say - maybe closeOnDelete is ok lol

[jcfranco]

Proceeding with closeOnDelete!