Skip to content

Commit 539aad1

Browse files
authored
chore: add process.env.NODE_ENV (#55)
1 parent bfdd4be commit 539aad1

File tree

5 files changed

+41
-33
lines changed

5 files changed

+41
-33
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ lib
2626
es
2727
yarn.lock
2828
package-lock.json
29+
pnpm-lock.yaml
2930
coverage/
3031
.doc
3132
dist/

package.json

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,29 +56,30 @@
5656
"@types/classnames": "^2.2.9",
5757
"@types/enzyme": "^3.10.8",
5858
"@types/jest": "^26.0.23",
59+
"@types/node": "^24.8.1",
5960
"@types/react": "^16.14.2",
6061
"@types/react-dom": "^16.9.10",
62+
"@typescript-eslint/eslint-plugin": "^5.59.0",
63+
"@typescript-eslint/parser": "^5.59.0",
6164
"@umijs/fabric": "^3.0.0",
62-
"glob": "^10.0.0",
6365
"cross-env": "^7.0.2",
6466
"dumi": "^2.0.0",
6567
"enzyme": "^3.0.0",
6668
"enzyme-adapter-react-16": "^1.0.1",
6769
"enzyme-to-json": "^3.4.0",
6870
"eslint": "^8.57.0",
71+
"eslint-plugin-jest": "^27.5.1",
72+
"eslint-plugin-unicorn": "^51.0.1",
6973
"father": "^4.0.0",
74+
"glob": "^10.0.0",
7075
"less": "^3.10.3",
7176
"np": "^7.0.0",
7277
"prettier": "^2.0.5",
7378
"rc-test": "^7.0",
7479
"react": "^16.0.0",
7580
"react-dom": "^16.0.0",
7681
"regenerator-runtime": "^0.13.7",
77-
"typescript": "~5.3.3",
78-
"@typescript-eslint/parser": "^5.59.0",
79-
"@typescript-eslint/eslint-plugin": "^5.59.0",
80-
"eslint-plugin-jest": "^27.5.1",
81-
"eslint-plugin-unicorn": "^51.0.1"
82+
"typescript": "~5.3.3"
8283
},
8384
"peerDependencies": {
8485
"react": ">=16.9.0",

src/Item.tsx

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,9 @@ function InternalItem<ItemType>(
6666

6767
// ================================ Render ================================
6868
const childNode =
69-
renderItem && item !== UNDEFINED ? renderItem(item, { index: order }) : children;
69+
renderItem && item !== UNDEFINED
70+
? renderItem(item, { index: order })
71+
: children;
7072

7173
let overflowStyle: React.CSSProperties | undefined;
7274
if (!invalidate) {
@@ -117,6 +119,9 @@ function InternalItem<ItemType>(
117119
}
118120

119121
const Item = React.forwardRef(InternalItem);
120-
Item.displayName = 'Item';
122+
123+
if (process.env.NODE_ENV !== 'production') {
124+
Item.displayName = 'Item';
125+
}
121126

122127
export default Item;

src/Overflow.tsx

Lines changed: 22 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,8 @@ export { OverflowContext } from './context';
1616

1717
export type { ComponentType } from './RawItem';
1818

19-
export interface OverflowProps<ItemType> extends Omit<React.HTMLAttributes<any>, 'prefix'> {
19+
export interface OverflowProps<ItemType>
20+
extends Omit<React.HTMLAttributes<any>, 'prefix'> {
2021
prefixCls?: string;
2122
className?: string;
2223
style?: React.CSSProperties;
@@ -377,26 +378,19 @@ function Overflow<ItemType = any>(
377378

378379
const mergedRenderRest = renderRest || defaultRenderRest;
379380

380-
const restNode = renderRawRest ? (
381-
<OverflowContext.Provider
382-
value={{
383-
...itemSharedProps,
384-
...restContextProps,
385-
}}
386-
>
387-
{renderRawRest(omittedItems)}
388-
</OverflowContext.Provider>
389-
) : (
390-
<Item
391-
{...itemSharedProps}
392-
// When not show, order should be the last
393-
{...restContextProps}
394-
>
395-
{typeof mergedRenderRest === 'function'
396-
? mergedRenderRest(omittedItems)
397-
: mergedRenderRest}
398-
</Item>
399-
);
381+
const restNode = renderRawRest ? (
382+
<OverflowContext.Provider
383+
value={{ ...itemSharedProps, ...restContextProps }}
384+
>
385+
{renderRawRest(omittedItems)}
386+
</OverflowContext.Provider>
387+
) : (
388+
<Item {...itemSharedProps} {...restContextProps}>
389+
{typeof mergedRenderRest === 'function'
390+
? mergedRenderRest(omittedItems)
391+
: mergedRenderRest}
392+
</Item>
393+
);
400394

401395
const overflowNode = (
402396
<Component
@@ -447,7 +441,9 @@ function Overflow<ItemType = any>(
447441
<ResizeObserver onResize={onOverflowResize} disabled={!shouldResponsive}>
448442
{overflowNode}
449443
</ResizeObserver>
450-
) : overflowNode;
444+
) : (
445+
overflowNode
446+
);
451447
}
452448

453449
const ForwardOverflow = React.forwardRef(Overflow);
@@ -465,11 +461,13 @@ type FilledOverflowType = ForwardOverflowType & {
465461
INVALIDATE: typeof INVALIDATE;
466462
};
467463

468-
ForwardOverflow.displayName = 'Overflow';
469-
470464
(ForwardOverflow as unknown as FilledOverflowType).Item = RawItem;
471465
(ForwardOverflow as unknown as FilledOverflowType).RESPONSIVE = RESPONSIVE;
472466
(ForwardOverflow as unknown as FilledOverflowType).INVALIDATE = INVALIDATE;
473467

468+
if (process.env.NODE_ENV !== 'production') {
469+
ForwardOverflow.displayName = 'Overflow';
470+
}
471+
474472
// Convert to generic type
475473
export default ForwardOverflow as unknown as FilledOverflowType;

src/RawItem.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,9 @@ const InternalRawItem = (props: RawItemProps, ref: React.Ref<any>) => {
4040
};
4141

4242
const RawItem = React.forwardRef(InternalRawItem);
43-
RawItem.displayName = 'RawItem';
43+
44+
if (process.env.NODE_ENV !== 'production') {
45+
RawItem.displayName = 'RawItem';
46+
}
4447

4548
export default RawItem;

0 commit comments

Comments
 (0)