Bad formatting on property pattern in a positional pattern when line wrapping

[kyoyama-kazusa]

Version Used:
VS 17.0 Preview 1.1

Steps to Reproduce:

Type code such as

OrderingSyntax former = orderings[i], latter = orderings[j];
if (
    (former, latter) is not (
        {
            Expression: IdentifierNameSyntax
            {
                Identifier: { ValueText: var formerIdentifier }
            }
        },
        {
            Expression: IdentifierNameSyntax
            {
                Identifier: { ValueText: var latterIdentifier }
            }
        }
    )
)
{
    continue;
}

...

Expected Behavior:

The format will be like as below:

OrderingSyntax former = orderings[i], latter = orderings[j];
if (
    (former, latter) is not (
        {
            Expression: IdentifierNameSyntax
            {
                Identifier: { ValueText: var formerIdentifier }
            }
        },
        {
            Expression: IdentifierNameSyntax
            {
                Identifier: { ValueText: var latterIdentifier }
            }
        }
    )
)
{
    continue;
}

Actual Behavior:
When click the button 'Fix Formatting', then we get

[sharwell]

Marking as help wanted, however it's my understanding that this is the expected behavior from the formatter (the indentation given above is one indent too much):

if (
    (former, latter) is not (
    {
        Expression: IdentifierNameSyntax
        {
            Identifier: { ValueText: var formerIdentifier }
        }
    },
    {
        Expression: IdentifierNameSyntax
        {
            Identifier: { ValueText: var latterIdentifier }
        }
    }
    )
)
{
    continue;
}

[kyoyama-kazusa]

@sharwell The inner property pattern should align as the same column as the bracket (former, latter):

// Indenting type 1 :)
(former, latter) is (
{
    property_pattern_clause
},
{
    another_property_pattern_clause
}
)

or append a indenting unit "\t" or " ":

// Indenting type 2 :)
(former, latter) is (
    {
        property_pattern_clause
    },
    {
        another_property_pattern_clause
    }
)

This way to indent code is also OK and can be acceptable. Both two ways above are better than the case from this issue mentioned:

// Malformed :(
    (former, latter) is (
{
    property_pattern_clause
},
{
    another_property_pattern_clause
}
    )

I prefer the second one (type 2), because the property pattern is surrounded by a pair or bracket (), which means the inner code should contain an extra indenting unit "\t" or " ".