File scoped namespaces

[AndrewKeepCoding]

Description

This PR will switch C# files from block-scoped namespaces to file-scoped namespaces.
Closes #1689.

Motivation and Context

Improves readability.

How Has This Been Tested?

• Confirmed that all unit tests passes.
• Run the app and navigated through all pages.

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

[ghost1372]

FYI @AndrewKeepCoding
Do we need all these commits? Use Visual Studio tools!

[AndrewKeepCoding]

I did. 🙂
However, I also reviewed all the files to check the changes, which is why there are separate commits. I've seen some cases that VS (or VS Code) doesn't apply this kind of changes correctly.

[Marv51]

Are you planning to add a .git-blame-ignore-revs file for this PR?

[AndrewKeepCoding]

@Marv51 You are right. Thanks.
TODO: Update the target commit id at the .git-blame-ignore-revs file.

[marcelwgn]

When creating a new UserControl, the namespace is before the usings. Is that behavior also the case for you @AndrewKeepCoding ?

[AndrewKeepCoding]

@marcelwgn No on my side. This is what I get:

using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Runtime.InteropServices.WindowsRuntime;
using Windows.Foundation;
using Windows.Foundation.Collections;
using Microsoft.UI.Xaml;
using Microsoft.UI.Xaml.Controls;
using Microsoft.UI.Xaml.Controls.Primitives;
using Microsoft.UI.Xaml.Data;
using Microsoft.UI.Xaml.Input;
using Microsoft.UI.Xaml.Media;
using Microsoft.UI.Xaml.Navigation;

// To learn more about WinUI, the WinUI project structure,
// and more about our project templates, see: http://aka.ms/winui-project-info.

namespace WinUIApp5
{
    public sealed partial class UserControl1 : UserControl
    {
        public UserControl1()
        {
            this.InitializeComponent();
        }
    }
}

[AndrewKeepCoding]

By the way, the item templates are being changed to file scoped namespaces in v1.7-exp2.

[marcelwgn]

Huh thats very interesting. Would be good to know why the results differ when creating a new UserControl. But that will be fixed with the new templates anyway (though I would have preferred the namespace to preceed usings but thats a different story).

This topic aside, looks like there are some merge conflicts which are not surprising though with these changes though.

[AndrewKeepCoding]

This topic aside, looks like there are some merge conflicts which are not surprising though with these changes though.

@marcelwgn Thanks! I fixed the conflicts.

[niels9001]

/azp run

[karkarl]

/azp run

[AndrewKeepCoding]

A review before another conflict would be greatly appreciated. 😅

[marcelwgn]

/azp run

[marcelwgn]

/azp run

[niels9001]

/azp run

[niels9001]

Merged! Thanks @AndrewKeepCoding !