Hook up the Save and Reset buttons

[carlos-zamora]

Summary of the Pull Request

Adds an event handler for the Save and Reset buttons. "Save" writes the settings to disk using the API introduced by #8018. "Reset" simply overwrites the settingsClone (what the Settings UI reads from) with settingsSource (provided by TermApp on Settings UI initialization).

References

#1564 - Settings UI

Validation Steps Performed

The following scenarios were tested and are verified to work properly:

• Change default profile in SUI, then save/reset
• Hide a profile, then save/reset
• Modifying the settings.json directly propagates changes to SUI
• "Reset" updates the current page

[carlos-zamora]

g, don't you need to init the profile list here? don't make a copy of this logic, unify it with the logic on lines 41-47 😬

@DHowett somehow you looked at the commit that isn't HEAD haha. The logic for refreshing the list of profiles was added in 23f08e8. I don't see how I can consolidate that with the constructor though. UpdateSettings needs to remove the list of profiles, then initialize it again. And the constructor has that InitializeComponent that I'm guessing we don't want to call in UpdateSettings.

[zadjii-msft]

I'd sign off if not for the one UnparsedDefaultProfile question

[ghost]

Hello @carlos-zamora!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.