Repository: apache/superset
File: superset-frontend/src/explore/components/controls/ConditionalFormattingControl/FormattingPopoverContent.tsx (TypeScript)

Code before:
        {showOperatorFields ? (
          (props: GetFieldValue) => renderOperatorFields(props, columnType)
        ) : (
          <Row gutter={12}>
            <Col span={6}>
              {renderOperator({ showOnlyNone: true, columnType })}
            </Col>
          </Row>
        )}
      </FormItem>
      <Row style={{ display: 'flex', height: '40px' }}>
        {showOperatorFields && showToAllRow && (
          <>
            <Col span={1}>
              <FormItem
                name="toAllRow"
                valuePropName="checked"
                initialValue={toAllRow}
              >
                <Checkbox
                  onChange={event => setToAllRow(event.target.checked)}
                  checked={toAllRow}
                />
              </FormItem>
            </Col>
            <Col style={{ padding: '5px' }}>
              <FormItem required label={t('Apply to entire row')} />
            </Col>
          </>
        )}
        {showOperatorFields && showToColorText && (
          <>
            <Col span={1}>
              <FormItem
                name="toTextColor"
                valuePropName="checked"
                initialValue={toTextColor}
              >
                <Checkbox
                  onChange={event => setToTextColor(event.target.checked)}
                  checked={toTextColor}
                />
              </FormItem>
            </Col>
            <Col style={{ padding: '5px' }}>
              <FormItem required label={t('Apply to text color')} />
            </Col>
          </>
        )}
      </Row>


Review comment (suggestion):
Same here... we shouldn't need to be changing the padding of Col components in multiple places. If it doesn't work as "vanilla" then we should see if the Col component itself need adjustment (or a new prop for certain contexts). 

Code after:
            />
          </FormItem>
        </Col>
      </Row>
      <FormItem noStyle shouldUpdate={shouldFormItemUpdate}>
        {showOperatorFields ? (
          (props: GetFieldValue) => renderOperatorFields(props, columnType)
        ) : (
          <Row gutter={12}>
            <Col span={6}>
              {renderOperator({ showOnlyNone: true, columnType })}
            </Col>
          </Row>
        )}
      </FormItem>
      <Row>
        {showOperatorFields && showToAllRow && (
          <Row gutter={20}>
            <Col span={1}>
              <FormItem
                name="toAllRow"
                valuePropName="checked"
                initialValue={toAllRow}
              >
                <Checkbox
                  onChange={event => setToAllRow(event.target.checked)}
                  checked={toAllRow}
                />
              </FormItem>
            </Col>
            <Col>
              <FormItem required>{t('To entire row')}</FormItem>
            </Col>
          </Row>
        )}
        {showOperatorFields && showToColorText && (
          <Row gutter={20}>
            <Col span={1}>
              <FormItem
                name="toTextColor"
                valuePropName="checked"
                initialValue={toTextColor}
              >
                <Checkbox
                  onChange={event => setToTextColor(event.target.checked)}
                  checked={toTextColor}
                />
              </FormItem>
            </Col>
            <Col>
              <FormItem required>{t('To text color')}</FormItem>