Skip to content

Commit

Permalink
fix(core): rename OnHighlightParams to OnActiveParams
Browse files Browse the repository at this point in the history
  • Loading branch information
francoischalifour committed Feb 4, 2021
1 parent 56cbb20 commit 2d7762d
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 9 deletions.
4 changes: 2 additions & 2 deletions packages/autocomplete-core/src/types/AutocompletePlugin.ts
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
import { AutocompleteScopeApi, BaseItem } from './AutocompleteApi';
import { AutocompleteOptions } from './AutocompleteOptions';
import { OnSelectParams, OnHighlightParams } from './AutocompleteSource';
import { OnSelectParams, OnActiveParams } from './AutocompleteSource';

type PluginSubscriber<TParams> = (params: TParams) => void;

interface PluginSubscribeParams<TItem extends BaseItem>
extends AutocompleteScopeApi<TItem> {
onSelect(fn: PluginSubscriber<OnSelectParams<TItem>>): void;
onActive(fn: PluginSubscriber<OnHighlightParams<TItem>>): void;
onActive(fn: PluginSubscriber<OnActiveParams<TItem>>): void;
}

export type AutocompletePlugin<
Expand Down
4 changes: 2 additions & 2 deletions packages/autocomplete-core/src/types/AutocompleteSource.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ export interface OnSelectParams<TItem extends BaseItem>
source: InternalAutocompleteSource<TItem>;
}

export type OnHighlightParams<TItem extends BaseItem> = OnSelectParams<TItem>;
export type OnActiveParams<TItem extends BaseItem> = OnSelectParams<TItem>;

export interface AutocompleteSource<TItem extends BaseItem> {
// This allows flavors to pass other keys to their source.
Expand Down Expand Up @@ -57,7 +57,7 @@ export interface AutocompleteSource<TItem extends BaseItem> {
* An item is highlighted either via keyboard navigation or via mouse over.
* You can trigger different behaviors based on the event `type`.
*/
onActive?(params: OnHighlightParams<TItem>): void;
onActive?(params: OnActiveParams<TItem>): void;
}

export type InternalAutocompleteSource<TItem extends BaseItem> = {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import { BaseItem } from './AutocompleteApi';
import { OnHighlightParams, OnSelectParams } from './AutocompleteSource';
import { OnActiveParams, OnSelectParams } from './AutocompleteSource';

export type AutocompleteSubscriber<TItem extends BaseItem> = {
onSelect(params: OnSelectParams<TItem>): void;
onActive(params: OnHighlightParams<TItem>): void;
onActive(params: OnActiveParams<TItem>): void;
};

export type AutocompleteSubscribers<TItem extends BaseItem> = Array<
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import {
AlgoliaInsightsHit,
InsightsApi,
InsightsClient,
OnHighlightParams,
OnActiveParams,
OnItemsChangeParams,
OnSelectParams,
} from './types';
Expand Down Expand Up @@ -59,7 +59,7 @@ export type CreateAlgoliaInsightsPluginParams = {
/**
* Hook to send an Insights event when an item is active.
*/
onActive?(params: OnHighlightParams): void;
onActive?(params: OnActiveParams): void;
};

export function createAlgoliaInsightsPlugin({
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ export type OnSelectParams = {
event: any;
};

export type OnHighlightParams = OnSelectParams;
export type OnActiveParams = OnSelectParams;

export type OnItemsChangeParams = {
insights: InsightsApi;
Expand Down

0 comments on commit 2d7762d

Please sign in to comment.