Lift button from the ThreadSettingsModal class component to the ConnectedThreadSettingsModal functional component.
Doing this to "stage" the lifting of JSX markup into separate steps to hopefully make it easier to review.
As part of the work to refactor ThreadSettingsModal... specifically to turn it into a functional component.
Depends on D3822
I think the wording here may be a bit confusing. In my experience, 'cancel' typically means close the alert, and there's another option like 'confirm' or something that will mean you're ok with discarding changes.
Here it seems like 'cancel' means we're ok with discarding changes. Not sure if it's just me though (cc @ted)