Skip to content

Commit

Permalink
Handle PR feedback.
Browse files Browse the repository at this point in the history
  • Loading branch information
atmgrifter00 committed Mar 8, 2024
1 parent 0f264d4 commit 123e3b4
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions packages/nimble-components/src/list-option/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import {
import { observable, attr } from '@microsoft/fast-element';
import { styles } from './styles';
import { template } from './template';
import type { DropdownOwner } from '../patterns/dropdown/types';
import type { ListOptionOwner } from '../patterns/dropdown/types';

declare global {
interface HTMLElementTagNameMap {
Expand Down Expand Up @@ -60,8 +60,8 @@ export class ListOption extends FoundationListboxOption {
}
}

private isDropdownOwner(parent: unknown): parent is DropdownOwner {
return typeof (parent as DropdownOwner).registerOption === 'function';
private isDropdownOwner(parent: unknown): parent is ListOptionOwner {
return typeof (parent as ListOptionOwner).registerOption === 'function';
}
}

Expand Down
2 changes: 1 addition & 1 deletion packages/nimble-components/src/patterns/dropdown/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,6 @@ export type DropdownAppearance =
* reactive form, where the form value is set to an option immediately after
* dynamically adding it.
*/
export interface DropdownOwner {
export interface ListOptionOwner {
registerOption: (option: ListOption) => void;
}
4 changes: 2 additions & 2 deletions packages/nimble-components/src/select/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ import {
} from '@microsoft/fast-web-utilities';
import { arrowExpanderDown16X16 } from '@ni/nimble-tokens/dist/icons/js';
import { styles } from './styles';
import { DropdownAppearance, DropdownOwner } from '../patterns/dropdown/types';
import { DropdownAppearance, ListOptionOwner } from '../patterns/dropdown/types';
import { errorTextTemplate } from '../patterns/error/template';
import type { ErrorPattern } from '../patterns/error/types';
import { iconExclamationMarkTag } from '../icons/exclamation-mark';
Expand Down Expand Up @@ -59,7 +59,7 @@ const isNimbleListOption = (el: Element): el is ListOption => {
*/
export class Select
extends FormAssociatedSelect
implements ErrorPattern, DropdownOwner {
implements ErrorPattern, ListOptionOwner {
@attr
public appearance: DropdownAppearance = DropdownAppearance.underline;

Expand Down

0 comments on commit 123e3b4

Please sign in to comment.