-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(input-time-zone): add max-items support #7705
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -80,6 +80,9 @@ export class InputTimeZone | |
*/ | ||
@Prop({ reflect: true }) form: string; | ||
|
||
/** Specifies the maximum number of input time zone options to display before displaying a scrollbar. */ | ||
@Prop({ reflect: true }) maxItems = 0; | ||
|
||
/** | ||
* Made into a prop for testing purposes only | ||
* | ||
|
@@ -301,6 +304,7 @@ export class InputTimeZone | |
disabled={this.disabled} | ||
label={this.messages.chooseTimeZone} | ||
lang={this.effectiveLocale} | ||
maxItems={this.maxItems} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. how will this prop be used? What happens when more items exist than the max allowed items? Will items simply not render or will the customer receive some kind of warning/alert that they have exceeded the max items? Will this be included in a separate PR? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Good questions. To summarize, this prop basically defines the amount of items displayed before a scrollbar is displayed. |
||
onCalciteComboboxBeforeClose={this.onComboboxBeforeClose} | ||
onCalciteComboboxBeforeOpen={this.onComboboxBeforeOpen} | ||
onCalciteComboboxChange={this.onComboboxChange} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WDYT about the following for doc consistency?: