Skip to content
This repository was archived by the owner on Sep 30, 2025. It is now read-only.

Commit 2ef82b5

Browse files
committed
update docs
1 parent c36df9e commit 2ef82b5

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

polaris-react/src/components/Popover/Popover.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,10 @@ export interface PopoverProps {
3333
preferredAlignment?: PopoverOverlayProps['preferredAlignment'];
3434
/** Show or hide the Popover */
3535
active: boolean;
36-
/** The element to activate the Popover */
36+
/** The element to activate the Popover.
37+
* If using a button, use the default or tertiary variant
38+
* which will show an active state when popover is active
39+
*/
3740
activator: React.ReactElement;
3841
/**
3942
* Use the activator's input element to calculate the Popover position

polaris.shopify.com/content/components/overlays/popover.mdx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ Popovers should:
5959
- Be used for secondary or less important information and actions since they’re hidden until merchants hit the trigger
6060
- Contain navigation or actions that share a relationships to each other
6161
- Be triggered by a clearly labeled button
62+
- Use a default or tertiary button as the activator
6263

6364
---
6465

0 commit comments

Comments
 (0)