Terminal: "Close all tabs?" dialog should highlight the "default" option

[zadjii-msft]

refer to screenshot:

Pressing enter here _closes the window_, because that's the focused/default button on this dialog. However, there's no way of knowing which is the selected button right now. We should either use the accent color with the button, or manually display the focus rect (is that even possible)?

Also for discussion: Should "close" be the default option here?

[DHowett-MSFT]

Gut feeling: cancel should be the default option.

[carlos-zamora]

Classic Edge:

Pressing Enter does trigger "Close all". Surprisingly, they don't make it clear that that's the default button here. We definitely should though.

On a separate note, consider adding that checkbox? (that should definitely be it's own issue though)

[mdtauk]

You could use the Accented Button style for the default option

[jantari]

My opinion is that you should use the focus rectangle.

When you only have two buttons and they are different colors it is still not clear which one is active.

[cinnamon-msft]

Along with this issue, the styling should be fixed as well to match the other dialog boxes.
Mockup proposal:

In terms of what should be default, I'd argue if we have the checkbox for "don't show this again", it should be Cancel. Otherwise, Close All Tabs should be default.

[msftbot[bot]]

:tada:This issue was addressed in #4401, which has now been successfully released as Windows Terminal Preview v0.9.433.0.:tada:

Handy links:

• Release Notes
• Store Download