Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
너무 멋져요! 고생하셨습니다~! |
codeit-kkm
left a comment
codeit-kkm
left a comment
There was a problem hiding this comment.
늦어서 죄송합니다 ! 수고하셨습니다!
| const observerRef = useRef<HTMLDivElement | null>(null); | ||
| const accessToken = localStorage.getItem("accessToken") ?? ""; | ||
| // const accessToken = localStorage.getItem("accessToken") ?? ""; | ||
| const accessToken = Cookies.get("accessToken") ?? ""; |
| import { twMerge } from "tailwind-merge"; | ||
| import clsx from "clsx"; | ||
| import { postImage } from "@/lib/apis/imageApi"; | ||
| import { HashLoader } from "react-spinners"; |
There was a problem hiding this comment.
Spinner 컴포넌트는 지피티로도 간단하게 만들 수 있어서 굳이 라이브러리로 안해도 될 거 같아요.
There was a problem hiding this comment.
오 라이브러리 사용보다 ai로 컴포넌트 만드는게 더 간단하게 할 수 있는 방법이군요! 참고하겠습니다🙏🏻
| color="#5534DA" | ||
| size={variant === "task" ? 30 : 50} | ||
| className="z-20" | ||
| /> |
There was a problem hiding this comment.
중첩 3항연산자를 사용하면 코드 가독성이 내려가요.
보통 삼항연산자는 해당 논리를 다 읽어야 이 코드가 어떻게 돌아가는지 이해가 가능한데 그 논리를 중첩으로 쓰면 더 이해하기 어렵겠죠?
그래서 이런경우에는 ts-pattern같은 라이브러리를 활용해보거나 함수로 분리하는 등 다른 방법을 고려해보는게 좋을 거 같아요.
There was a problem hiding this comment.
토스 ts-pattern
관련 글 찾아서 읽어봤는데 처음보는 방법이라 흥미롭네요! 견해를 넓혀주셔서 감사합니다😆
| ); | ||
| setImageUrl(imageUrl); | ||
| } catch (error) { | ||
| console.error("이미지 업로드 에러:", error); |
There was a problem hiding this comment.
이런부분 console이 아니라 Toast로 에러메시지 띄어주면 유저가 더 이해하기 수월하겠죠?
toast.open("이미지 업로드 에러")There was a problem hiding this comment.
토스트 알림을 사용하면 유저 경험이 확실히 좋아질것 같다고 느꼈습니다.🥲 우선 해당 페이지 일관성을 위해서 콘솔로 에러처리 해뒀는데 다시 리팩토링 해보겠습니다 의견주셔서 너무 감사드려요!👍🏻
5 participants
#️⃣ Issue Number
#133
📝 요약
🛠️ PR 유형
📷 스크린샷
2025-04-16.221100.mp4
📢 공유 사항
https://www.npmjs.com/package/react-spinners?activeTab=readme
리액트 스피너를 사용했습니다. npm install 필요합니다.
📚 참고 자료
https://velog.io/@scy512/REACT-SPINNERS