ClipboardCopy extra commas

[jschuler]

ClipboardCopy component renders unnecessary commas between text elements.
Example:

import React from 'react';
import { ClipboardCopy } from '@patternfly/react-core';

const someValue = "hello world";
const Basic = () => <ClipboardCopy hoverTip="Copy" clickTip="Copied">
This is editable {someValue}
{' '}
test
</ClipboardCopy>;

[DaoDaoNoCode]

@jschuler I think this is because the props.children will be a string array in this way, one workaround could be setting the children to a string:

{`This is editable ${someValue}
${' '}
test`}

[DaoDaoNoCode]

This ClipboardCopy currently accepts the type ReactNode as its children type based on the doc, is it purposed to do so? I think it would be better to limit it to the string | number because there is no point to render a non-text node to a ClipboardCopy. Any thoughts? @tlabaj @mcarrano

[tlabaj]

@DaoDaoNoCode I agree, string or number would be more appropriate type. However, that would be. breaking change. Can we fix this with current implementation and perhaps open ac breaking change issue?