default <Popover> tag should be fragment? #1121
Replies: 3 comments
-
Hello, im sorry i wont answer your question but rather ask you a question: where did you see an actual popup component? I only find 4 components listed on the github project page... |
Beta Was this translation helpful? Give feedback.
-
@Jbmanllr check out the |
Beta Was this translation helpful? Give feedback.
-
Hey @ianstormtaylor! The main reason the Not always needed, but it's a fairly common approach, which is why we've opted to have a Since the Going to convert this to a discussion 👍 |
Beta Was this translation helpful? Give feedback.
-
Just an idea for an improvement. As far as I know a popover is most commonly a button (or some other click/hover target) and a panel:
But the current implementation has
<div>
as the default element for the root<Popover>
component. I think the default should beReact.Fragment
instead so that no wrapper is rendered?Beta Was this translation helpful? Give feedback.
All reactions