Remove default terminal item from SUI

[cinnamon-msft]

Looks like we forgot to comment out the Default terminal setting from the UI.

Commented it out here.

[zadjii-msft]

You gotta run runxamlformat now to format XAML files 😉

[DHowett]

Just format it!

[miniksa]

SOMEONE promised me that x:Load="False" was good enough here. :|

[zadjii-msft]

SOMEONE promised me that x:Load="False" was good enough here. :|

Wait it totally should have been. Why isn't it?

[DHowett]

Yeah, I totally don't understand why it wasn't sufficient. Are we calling FindElement on it? o_O

[DHowett]

OH. It's inside a SettingContainer. The setting container loads it immediately because it is a data template(!). If the x:Load was on the SettingContainer, the whole element would be kill.

[miniksa]

OH. It's inside a SettingContainer. The setting container loads it immediately because it is a data template(!). If the x:Load was on the SettingContainer, the whole element would be kill.

DERP. Sorry.

[DHowett]

@msftbot merge this in 2 minutes ago

[ghost]

Hello @DHowett!

Because you've given me some instructions on how to help merge this pull request, I'll be modifying my merge approach. Here's how I understand your requirements for merging this pull request:

• I won't merge this pull request until after the UTC date Wed, 31 Mar 2021 18:37:15 GMT, which is in 2 minutes

If this doesn't seem right to you, you can tell me to cancel these instructions and use the auto-merge policy that has been configured for this repository. Try telling me "forget everything I just told you".