You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Use of Layout options in PdmUiOrdering is difficult and the default constructor hides the names of the struct members. This makes the client code difficult to read and understand.
Suggestion
Move MAX_COLUMN_SPAN to PdmUiOrdering
Avoid default constructor, add initializer
static const int MAX_COLUMN_SPAN = -1;
struct LayoutOptions
{
bool newRow {true};
int totalColumnSpan{MAX_COLUMN_SPAN};
int leftLabelColumnSpan{MAX_COLUMN_SPAN};
};
The text was updated successfully, but these errors were encountered:
magnesj
added
the
System
Refactoring or build/test system issues. Code improvments not directly affecting the user.
label
Nov 8, 2023
Use of Layout options in PdmUiOrdering is difficult and the default constructor hides the names of the struct members. This makes the client code difficult to read and understand.
Suggestion
Move
MAX_COLUMN_SPAN
to PdmUiOrderingAvoid default constructor, add initializer
The text was updated successfully, but these errors were encountered: