Remove needToAddSubGroup bool, use presence of content as indicator

This commit is contained in:
John Lewin 2017-12-27 07:58:21 -08:00
parent 6c5d3bda14
commit 0c3f3a94b1

View file

@ -269,14 +269,11 @@ namespace MatterHackers.MatterControl.SlicerConfiguration
var subGroupLayoutTopToBottom = new FlowLayoutWidget(FlowDirection.TopToBottom);
subGroupLayoutTopToBottom.AnchorAll();
bool needToAddSubGroup = false;
foreach (OrganizerSubGroup subGroup in group.SubGroupsList)
{
var section = AddSettingRowsForSubgroup(subGroup, oemAndUserContext, showHelpControls);
if (section != null)
{
needToAddSubGroup = true;
var groupBox = new AltGroupBox(subGroup.Name.Localize())
{
TextColor = ActiveTheme.Instance.PrimaryTextColor,
@ -291,7 +288,7 @@ namespace MatterHackers.MatterControl.SlicerConfiguration
}
}
if (needToAddSubGroup)
if (subGroupLayoutTopToBottom.Children.Count > 0)
{
var scrollOnGroupTab = new SliceSettingListControl();
subGroupLayoutTopToBottom.VAnchor = VAnchor.Fit;