Use standard Expander control for review setup page instead of SettingsExpander

[florelis]

Summary of the pull request

This changes the SettingsExpander control we were using in the review page of the setup flow for a standard Expander control

References and relevant issues

Detailed description of the pull request / Additional comments

There seems to be a bug in the SettingsExpander where it crashes when expanding if the items contained are not SettingsCards (but only when running without a debugger). We had a NavigationPane, so we were being hit by this.

To avoid this, we use a standard Expander control. The downside is that we have to re-do some of the styling manually, as the settings controls already had a good default style.

We could have used a SettingsCard to contain the NavigationPane, but these are intended to be used with a "header" in the left and a "content" in the right, but we wanted to use the full width for the pane.

Validation steps performed

PR checklist

• Closes #xxx
• Tests added/passed
• Documentation updated

[florelis]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).