Delete comment icon is very close to close window icon

[jcansdale]

• GitHub Extension for Visual Studio version: 2.5.4

The ✖️ icon to delete an inline-comment is very close to the icon used to close the inline-comment window and also appears in the top right hand corner of a box.

If a user accidentally clicks on the delete icon, there is no confirmation to correct their mistake.

Maybe we could change it to be a ... drop down menu with a Delete option like .com relatively recently changed to.

[StanleyGoldman]

Related to #1713

[donokuda]

I'm going to take a quick stab at this and #1713.

I'll focus on two things:

1. Increasing the spacing between the ✏️ and the ❌ octicon
2. Popup a dialog when a user clicks on the ❌ asking them to confirm if they want to delete the comment. (and clicking "Delete" should delete the comment.)

EDIT: Side goal, regardless if my pull request is merged or not, is that I want to learn how to implement a modal dialog. I think there are places in the extension where we could utilize this pattern.

[meaghanlewis]

I think we can close this out now. Please re-open if I'm incorrect 😄