Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨ Feat: 로딩 스피너 #58

Merged
merged 5 commits into from
May 11, 2024
Merged

✨ Feat: 로딩 스피너 #58

merged 5 commits into from
May 11, 2024

Conversation

JayChae
Copy link
Contributor

@JayChae JayChae commented May 10, 2024

🧩 이슈 번호

✅ 작업 사항

로딩 스피너

spinner.mp4

test 페이지에서 확인 가능,

list 페이지에서는

<div className={styles.loading_spinner}>
    <LoadingSpinner />
</div>

<div className={styles.moreButton}>
    <CustomButton btnText={<LoadingSpinner />} disabled={true} />
</div>

이렇게 사용했습니다.

👩‍💻 공유 포인트 및 논의 사항

이렇게 하는 게 맞나?
컴포넌트로 적절한지 의견 좀 주세요.

@JayChae JayChae added the ✨ Feat 기능 개발 label May 10, 2024
@JayChae JayChae self-assigned this May 10, 2024
Copy link

🎉 구현한 기능 Preview: https://fandom-oz9mvww6h-easyhyun00s-projects.vercel.app

Copy link
Contributor

@easyhyun00 easyhyun00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!! 컴포넌트로 적절하다고 생각이 듭니다
근데 버튼 안에 있을 땐 색이 조금 안 어울리네요ㅋㅋㅋㅋㅋ버튼 안에서는 하얀색or검은색이 잘 어울릴 거 같아요
혹시 svg 로 하면 색상 수정할 수 있지 않을까요???

Comment on lines 4 to 6
const LoadingSpinner = () => {
return <img src={spinner} style={{ height: '100%' }} />;
};
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

props로 크기 같은 거 주는 거 있으면 좋을 거 같아요

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

width랑 height 주는 방식으로 해보겠습니다

<LoadingSpinner />
</div>
<div className={styles.moreButton}>
<CustomButton btnText={<LoadingSpinner />} disabled={true} />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

boolean 값으로 props 넘기는건 true 일 때 그냥 아래처럼 적으셔도 됩니다!

<CustomButton btnText={<LoadingSpinner />} disabled />

@JayChae
Copy link
Contributor Author

JayChae commented May 11, 2024

@easyhyun00

default.mp4

svg 컴포넌트로 수정하니까, 따로 컴포넌트는 필요 없을 것 같아요! svg 컴포넌트로 넓이 높이 색 다 지정할 수 있으니까 더 좋은 것 같네요. 더보기 버튼은 제가 볼 땐 흰색이 좋은 것 같은데 어떤가요

@easyhyun00
Copy link
Contributor

@credit 오 완전좋아요 그리고 gif 보다 화질이 조금 더 좋아보이네요

Copy link

🎉 구현한 기능 Preview: https://fandom-2nxmfn1g9-easyhyun00s-projects.vercel.app

@JayChae JayChae merged commit 3c72ef1 into main May 11, 2024
1 check passed
@JayChae JayChae deleted the feat-57 branch May 11, 2024 07:43
easyhyun00 added a commit to easyhyun00/fandom-k that referenced this pull request May 11, 2024
* 📦️ Chore: 로딩스피너 세팅

* ✨ Feat: 로딩 스피너 적용

* 🔨 Refactor: 스피너 주석

* ✨ Feat: svg 컴포넌트로 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ Feat: 로딩 컴포넌트 개발
2 participants