Skip to content

Commit 2132315

Browse files
chore(picker): add/correct arg naming
- adds missing placeholder arg to template - corrects content arg to popoverContent
1 parent 7d09a19 commit 2132315

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

components/picker/stories/picker.stories.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ export default {
9797
},
9898
isHovered,
9999
isActive,
100-
content: { table: { disable: true } },
100+
popoverContent: { table: { disable: true } },
101101
},
102102
args: {
103103
rootClass: "spectrum-Picker",

components/picker/stories/template.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -109,6 +109,7 @@ export const Template = ({
109109
size = "m",
110110
label,
111111
labelPosition = "top",
112+
placeholder,
112113
helpText,
113114
isQuiet = false,
114115
isOpen = false,
@@ -125,7 +126,7 @@ export const Template = ({
125126
// Helps ensure that Popover appears below the Picker, with side labels layout.
126127
display: "block",
127128
},
128-
content = [],
129+
popoverContent = [],
129130
} = {}, context = {}) => {
130131
const pickerMarkup = Picker({
131132
size,
@@ -134,17 +135,18 @@ export const Template = ({
134135
isInvalid,
135136
isDisabled,
136137
isLoading,
137-
content,
138+
placeholder,
139+
popoverContent,
138140
labelPosition,
139141
ariaLabeledBy: fieldLabelId,
140142
}, context);
141143

142-
const popoverMarkup = content.length !== 0 ? Popover({
144+
const popoverMarkup = popoverContent.length !== 0 ? Popover({
143145
isOpen: isOpen && !isDisabled && !isLoading,
144146
withTip: false,
145147
position: "bottom",
146148
isQuiet,
147-
content,
149+
content: popoverContent,
148150
size,
149151
customStyles: customPopoverStyles,
150152
}, context) : "";

0 commit comments

Comments
 (0)