Documentation Incorrect

[HeinzKessler]

The statement under "Remarks" is not correct: If the values for Top, Bottom, Left, or Rights were set separately, then All returns -1, even if all 4 properties have the same value:
var p = new Padding( 5 );
// Here, All returns 5
p.Left = 5;
// Now, All returns -1, although all 4 properties still have a value of 5
p.All = 5;
// Now, All returns 5 again

see also: https://referencesource.microsoft.com/#System.Windows.Forms/winforms/Managed/System/WinForms/Padding.cs,6c9dc153b2c496ae

---

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

• ID: 0c67d24a-2485-92bb-9e66-87104cf3eb61
• Version Independent ID: 9e56ed04-9020-f76c-a783-26ac11dfbee6
• Content: Padding.All Property (System.Windows.Forms)
• Content Source: xml/System.Windows.Forms/Padding.xml
• Service: unspecified
• GitHub Login: @dotnet-bot
• Microsoft Alias: dotnetcontent

[pkulikov]

I wonder if it's a bug in documentation or implementation?
Setter of the Left property:

if (_all || _left != value) {
    _all = false;
    _left = value;
}

can be improved like this, for example:

if (_left == value) return;
_left = value;
_all = _top == _left && _top == _right && _top == _bottom;

[HeinzKessler]

That's right: The implementation is wrong, but I don't think that they will change it, because thats would break people's existing code.

[mairaw]

@Tanya-Solyanik can you comment on this one? There's an indication that this is a product bug, but let me know if you'd like to just document the behavior as-is.

[Tanya-Solyanik]

This might be a product bug, but, like HeinzKessler said, it would be a breaking change to fix this and the workaround is simple. As is 'Padding.Allgetter indicates that padding values were set using thePadding.All` setting. this information might be useful. @mairaw - could you please document that?

[pkulikov]

As is Padding.All getter indicates that padding values were set using the Padding.All setting.

Or during construction. Both by the one-parameter constructor and by the four-parameter constructor if it gets all four the same values: Padding.All property is also defined (as I read it from the code).