Dayong Lee

Results 67 comments of Dayong Lee

```js 1 2 3 ``` You have to write like this. ```js // pagination.tsx const PaginationLink = ({ className, isActive, size = 'icon', ...props }: PaginationLinkProps) => ( ) ```...

https://nextjs.org/docs/app/api-reference/next-config-js/optimizePackageImports next.js์—์„œ๋Š” lucide-react library๋ฅผ ๋งŽ์ด ์“ฐ๋Š” ๊ฒƒ ๊ฐ™์•„์„œ ๋‹ค ๋ฐ”๊ฟจ๋Š”๋ฐ 1B์”ฉ ์ •๋„๋ฐ–์— ์ฐจ์ด๊ฐ€ ์•ˆ๋‚ฉ๋‹ˆ๋‹ค! ๊ทธ๋ž˜์„œ ์ฐพ์•„๋ณด๋‹ˆ๊น next.js์—์„œ react-icons ๋ผ์ด๋ธŒ๋Ÿฌ๋ฆฌ๋Š” ๊ธฐ๋ณธ์œผ๋กœ ์ตœ์ ํ™”ํ•˜๊ณ  ์žˆ์—ˆ๋„ค์š”.. @dotoleeoak

์ ์šฉ ํ›„ ๋นŒ๋“œ ๋กœ๊ทธ ์บก์ฒ˜ ์‚ฌ์ง„์ž…๋‹ˆ๋‹ค!

![Screenshot 2024-01-24 at 4 51 07โ€ฏPM](https://github.com/skkuding/codedang/assets/50468628/aa3496b7-b511-4638-8ac0-622c4f9636d3) ์ผ๋‹จ ๋ฐ”๊พธ๊ณ  ๋นŒ๋“œํ•˜๋ฉด icon ๊ด€๋ จ ๋ผ์ด๋ธŒ๋Ÿฌ๋ฆฌ๊ฐ€ ํฐ ๋น„์ค‘์„ ์ฐจ์ง€ ํ•˜์ง€ ์•Š์•„๋ณด์ด๋Š”๋ฐ ์•„๋งˆ `lucide-react`๋Š” ๊ทธ๋ƒฅ js๋กœ ๋นŒ๋“œํ•˜์ง€ ์•Š๋Š” ๊ฒŒ ์•„๋‹๊นŒ ์‹ถ์Šต๋‹ˆ๋‹ค!

@Kohminchae, @EliseCho13 ์ถ”๊ฐ€ ์˜๊ฒฌ ์žˆ์œผ๋ฉด ๋‚จ๊ฒจ์ฃผ์„ธ์š”! ๋”ฐ๋กœ ์ด๊ฒฌ ์—†์œผ๋ฉด shadcn์œผ๋กœ ์ง„ํ–‰ํ•˜๊ฒ ์Šต๋‹ˆ๋‹ค!

์ด๊ฑฐ ๋ˆ„๊ฐ€ ์•ˆํ•˜๋ฉด ์ด๋”ฐ๊ฐ€ ํšŒ์˜ ๋•Œ ์ƒํ™ฉ ์„ค๋ช… ๋“ฃ๊ณ  ์ œ๊ฐ€ ํ•ด๋„ ๋ ๊นŒ์š”?

We already have a native api(Array.prototype.join), so I thinks that it needs only in es-toolkit/compat.

You can see a same case in [here(`es-toolkit/compat/concat`)](https://es-toolkit.slash.page/reference/compat/array/concat.html). > This function is only available in es-toolkit/compat for compatibility reasons. It either has alternative native JavaScript APIs or isnโ€™t fully optimized...

I think that you don't have to close this PR at now, it is just my opinion. Maybe, you decide whether close or not after reviewing @raon0211. Thanks!