Why not save one more line with file-scoped namespaces?

[guardrex]

Summary

Couldn't the framework (and doc examples) save one more vertical line with file-scoped namespaces by collapsing the namespace line into any lines at the top of the file (e.g., make it the last line or the first line)?

Motivation and goals

My understanding is that one of the benefits of this feature is to vertically collapse code. As implemented today, it only saves one line that doesn't affect scrolling. Only the final brace } line is dropped/saved.

Risks / unknowns

I think the risk of disorganization is low.

Example

src/ProjectTemplates/Web.ProjectTemplates/content/ComponentsWebAssembly-CSharp/Server/Pages/Error.cshtml.cs:

using System.Diagnostics;
using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.RazorPages;

namespace ComponentsWebAssembly_CSharp.Server.Pages;

[ResponseCache(Duration = 0, Location = ResponseCacheLocation.None, NoStore = true)]
[IgnoreAntiforgeryToken]
public class ErrorModel : PageModel
{
    ...
}

... TO ...

using System.Diagnostics;
using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.RazorPages;
namespace ComponentsWebAssembly_CSharp.Server.Pages;

[ResponseCache(Duration = 0, Location = ResponseCacheLocation.None, NoStore = true)]
[IgnoreAntiforgeryToken]
public class ErrorModel : PageModel
{
    ...
}

... OR with the namespace at the top of the file ...

namespace ComponentsWebAssembly_CSharp.Server.Pages;
using System.Diagnostics;
using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.RazorPages;

[ResponseCache(Duration = 0, Location = ResponseCacheLocation.None, NoStore = true)]
[IgnoreAntiforgeryToken]
public class ErrorModel : PageModel
{
    ...
}

🤔

I kind'a like it at the top of the file. That locates it quickly at a glance 👀, which is all that placing it on a line with vertical space around it seems to accomplish.

[Tratcher]

My understanding is that one of the benefits of this feature is to vertically collapse code.

The main benefit was to reduce a level of indentation and braces. e.g. horizontal space, not vertical.

[guardrex]

I thought I read somewhere something about reduced scrolling.

No worries. Just thought that I'd ask.

[Tratcher]

Yes, horizontal scrolling 😁

[guardrex]

btw ... I did find it (emphasis added) ...

This affects clarity, uses horizontal and vertical space, and is often unsatisfying for users both used to C# and coming from other languages (which commonly have less ceremony here).

Reference: https://learn.microsoft.com/en-us/dotnet/csharp/language-reference/proposals/csharp-10.0/file-scoped-namespaces#motivation

[Tratcher]

Ha, nice. It does get rid of a pair of braces so there's a little vertical improvement. The horizontal is more impactful in my experience though.

[guardrex]

Besides the one-line/two-line saving situation, it seems strange to me to hang the line out in space like that for no other reason than perhaps to be able to identify the namespace quickly. Seems like if that's desired, it would just be the first line of the file.

It's cool. I just started rolling this into Blazor docs today, and I have a goal now of at least getting all of the 7.0 docs to adopt it. I could go back to 6.0 (C# 10), but naaaaaah ... too many other priorities.