Skip to content

Commit

Permalink
Fix primefaces#3302: Multiselect add overlayVisible property
Browse files Browse the repository at this point in the history
  • Loading branch information
melloware committed Nov 25, 2022
1 parent 6a2e29a commit 2543eaf
Show file tree
Hide file tree
Showing 4 changed files with 62 additions and 42 deletions.
6 changes: 6 additions & 0 deletions api-generator/components/multiselect.js
Original file line number Diff line number Diff line change
Expand Up @@ -286,6 +286,12 @@ const MultiSelectProps = [
type: 'boolean',
default: 'false',
description: 'Defaults to the option itself as the value of an option. Overrides the optionValue prop.'
},
{
name: 'overlayVisible',
type: 'boolean',
default: 'false',
description: 'Specifies the visibility of the overlay panel.'
}
];

Expand Down
6 changes: 6 additions & 0 deletions components/doc/multiselect/apidoc.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,12 @@ export function ApiDoc(props) {
<td>null</td>
<td>Inline style of the overlay panel element.</td>
</tr>
<tr>
<td>overlayVisible</td>
<td>boolean</td>
<td>false</td>
<td>Specifies the visibility of the overlay panel.</td>
</tr>
<tr>
<td>scrollHeight</td>
<td>string</td>
Expand Down
7 changes: 7 additions & 0 deletions components/lib/multiselect/MultiSelect.js
Original file line number Diff line number Diff line change
Expand Up @@ -534,6 +534,12 @@ export const MultiSelect = React.memo(
ObjectUtils.combinedRefs(inputRef, props.inputRef);
}, [inputRef, props.inputRef]);

React.useEffect(() => {
setTimeout(() => {
props.overlayVisible ? show() : hide();
}, 100);
}, [props.overlayVisible]);

useUpdateEffect(() => {
if (overlayVisibleState && hasFilter) {
alignOverlay();
Expand Down Expand Up @@ -685,6 +691,7 @@ MultiSelect.defaultProps = {
optionLabel: null,
optionValue: null,
options: null,
overlayVisible: false,
panelClassName: null,
panelFooterTemplate: null,
panelHeaderTemplate: null,
Expand Down
85 changes: 43 additions & 42 deletions components/lib/multiselect/multiselect.d.ts
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import * as React from 'react';
import TooltipOptions from '../tooltip/tooltipoptions';
import { CSSTransitionProps } from '../csstransition';
import { SelectItemOptionsType } from '../selectitem/selectitem';
import TooltipOptions from '../tooltip/tooltipoptions';
import { IconType } from '../utils';
import { VirtualScrollerProps } from '../virtualscroller';
import { SelectItemOptionsType } from '../selectitem/selectitem';

type MultiSelectOptionGroupTemplateType = React.ReactNode | ((option: any, index: number) => React.ReactNode);

Expand Down Expand Up @@ -74,64 +74,65 @@ interface MultiSelectFilterOptions {
}

export interface MultiSelectProps extends Omit<React.DetailedHTMLProps<React.InputHTMLAttributes<HTMLDivElement>, HTMLDivElement>, 'onChange' | 'ref'> {
appendTo?: MultiSelectAppendToType;
ariaLabelledBy?: string;
children?: React.ReactNode;
className?: string;
dataKey?: string;
disabled?: boolean;
display?: MultiSelectDisplayType;
dropdownIcon?: IconType<MultiSelectProps>;
emptyFilterMessage?: MultiSelectEmptyFilterMessageType;
filter?: boolean;
filterBy?: string;
filterLocale?: string;
filterMatchMode?: string;
filterPlaceholder?: string;
filterTemplate?: MultiSelectFilterTemplateType;
fixedPlaceholder?: boolean;
id?: string;
inputId?: string;
inputRef?: React.Ref<HTMLSelectElement>;
itemTemplate?: MultiSelectItemTemplateType;
maxSelectedLabels?: number;
name?: string;
value?: any;
options?: SelectItemOptionsType;
optionLabel?: string;
optionValue?: string;
optionDisabled?: MultiSelectOptionDisabledType;
optionGroupLabel?: string;
optionGroupChildren?: string;
optionGroupLabel?: string;
optionGroupTemplate?: MultiSelectOptionGroupTemplateType;
display?: MultiSelectDisplayType;
style?: React.CSSProperties;
className?: string;
optionLabel?: string;
optionValue?: string;
options?: SelectItemOptionsType;
overlayVisible?: boolean;
panelClassName?: string;
panelFooterTemplate?: MultiSelectPanelFooterTemplateType;
panelHeaderTemplate?: MultiSelectPanelHeaderTemplateType;
panelStyle?: React.CSSProperties;
virtualScrollerOptions?: VirtualScrollerProps;
scrollHeight?: string;
placeholder?: string;
fixedPlaceholder?: boolean;
disabled?: boolean;
showClear?: boolean;
filter?: boolean;
filterBy?: string;
filterMatchMode?: string;
filterPlaceholder?: string;
filterLocale?: string;
emptyFilterMessage?: MultiSelectEmptyFilterMessageType;
removeIcon?: IconType<MultiSelectProps>;
resetFilterOnHide?: boolean;
scrollHeight?: string;
selectAll?: boolean;
selectedItemTemplate?: MultiSelectSelectedItemTemplateType;
selectedItemsLabel?: string;
selectionLimit?: number;
showClear?: boolean;
showSelectAll?: boolean;
style?: React.CSSProperties;
tabIndex?: number;
dataKey?: string;
inputId?: string;
appendTo?: MultiSelectAppendToType;
tooltip?: string;
tooltipOptions?: TooltipOptions;
maxSelectedLabels?: number;
selectionLimit?: number;
selectedItemsLabel?: string;
ariaLabelledBy?: string;
itemTemplate?: MultiSelectItemTemplateType;
filterTemplate?: MultiSelectFilterTemplateType;
selectedItemTemplate?: MultiSelectSelectedItemTemplateType;
panelHeaderTemplate?: MultiSelectPanelHeaderTemplateType;
panelFooterTemplate?: MultiSelectPanelFooterTemplateType;
transitionOptions?: CSSTransitionProps;
dropdownIcon?: IconType<MultiSelectProps>;
removeIcon?: IconType<MultiSelectProps>;
showSelectAll?: boolean;
selectAll?: boolean;
useOptionAsValue?: boolean;
value?: any;
virtualScrollerOptions?: VirtualScrollerProps;
onBlur?(event: React.FocusEvent<HTMLInputElement>): void;
onChange?(e: MultiSelectChangeParams): void;
onFilter?(e: MultiSelectFilterParams): void;
onFocus?(event: React.FocusEvent<HTMLInputElement>): void;
onBlur?(event: React.FocusEvent<HTMLInputElement>): void;
onShow?(): void;
onHide?(): void;
onFilter?(e: MultiSelectFilterParams): void;
onSelectAll?(e: MultiSelectAllParams): void;
children?: React.ReactNode;
useOptionAsValue?: boolean;
onShow?(): void;
}

export declare class MultiSelect extends React.Component<MultiSelectProps, any> {
Expand Down

0 comments on commit 2543eaf

Please sign in to comment.