flutter: skip gen Container with empty props - #192

Thanks!!

It is probably better to do like props =..., then isEmptyProps() else...

Extract props to a separate variable? Gotcha

@bernaferrari done

Is empty props working as it should? Because the width/height already check for zero and ignore that. Not sure the difference

yup, this function checks all properties and skips if property is default which resulted in unnecessary container.

const hasEmptyProps = (props: Record<string, string>): boolean => {
...
}

currently these props are being checked for default values

properties.width = skipDefaultProperty(width, "0");
properties.height = skipDefaultProperty(height, "0");
properties.padding = propPadding;
properties.clipBehavior = clipBehavior;

const parsedDecoration = skipDefaultProperty(
  propBoxDecoration,
  "BoxDecoration()",
);
properties.decoration = clipBehavior ? propBoxDecoration : parsedDecoration;

const isEmptyProps = hasEmptyProps(properties);

without this check result will be

Before Patch

Container(
	child: Column(
		children: [SomeWidgetsHere()...]
	)
)

After Patch

Column(
	children: [SomeWidgetsHere()...]
)