Skip to content

Commit b7071c9

Browse files
committed
chore: Split maxCount logic alone with responsive
1 parent 582ae7a commit b7071c9

File tree

2 files changed

+9
-7
lines changed

2 files changed

+9
-7
lines changed

examples/basic.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ function renderItem(item: ItemType) {
3232
}
3333

3434
const Demo = () => {
35-
const [responsive, setResponsive] = React.useState(true);
35+
const [responsive, setResponsive] = React.useState(false);
3636
const [data, setData] = React.useState(createData(5));
3737

3838
return (
@@ -43,7 +43,7 @@ const Demo = () => {
4343
setResponsive(!responsive);
4444
}}
4545
>
46-
Responsive: {String(responsive)}
46+
{responsive ? 'Responsive' : 'MaxCount: 6'}
4747
</button>
4848
<select
4949
style={{ width: 200, height: 32 }}

src/Item.tsx

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export interface ItemProps<ItemType> {
1111
itemKey?: React.Key;
1212
registerSize: (key: React.Key, width: number) => void;
1313
children?: React.ReactNode;
14-
display?: boolean;
14+
display: boolean;
1515
order: number;
1616
}
1717

@@ -25,10 +25,12 @@ export default function Item<ItemType>(props: ItemProps<ItemType>) {
2525
itemKey,
2626
className,
2727
children,
28-
display = true,
28+
display,
2929
order,
3030
} = props;
3131

32+
const mergedHidden = responsive && !display;
33+
3234
// ================================ Effect ================================
3335
function internalRegisterSize(width: number) {
3436
registerSize(itemKey!, width);
@@ -48,11 +50,11 @@ export default function Item<ItemType>(props: ItemProps<ItemType>) {
4850
<div
4951
className={classNames(prefixCls, className)}
5052
style={{
51-
opacity: display ? 1 : 0.2,
52-
height: display ? undefined : 0,
53+
opacity: mergedHidden ? 0.2 : 1,
54+
height: mergedHidden ? 0 : undefined,
5355
overflowY: responsive ? 'hidden' : undefined,
5456
order: responsive ? order : undefined,
55-
pointerEvents: 'none',
57+
pointerEvents: mergedHidden ? 'none' : undefined,
5658
}}
5759
>
5860
{childNode}

0 commit comments

Comments
 (0)