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

[GGFE-225] 확성기 개수 제한 없이 작동하게 css 변경 #970

Merged
merged 6 commits into from
Sep 5, 2023

Conversation

PHJoon
Copy link
Contributor

@PHJoon PHJoon commented Aug 31, 2023

📌 개요

  • 확성기 개수 제한 없이 작동하는 로직으로 변경

💻 작업사항

  • 기존에 확성기 10개로 생각하고 작성하였던 확성기 css를 최대 1000개까지 받을 수 있게 변경했습니다.
  • 1000개는 임의로 설정한 값이라 더 큰 값이 필요하다면 바꾸기만 하면 됩니다.
  • 피드백 주신 내용 반영하여 div에 style 직접 주는 방식으로 변경했습니다!
  • megaphone 기본값을 관리자 멘트 -> 관리자 + 아이템 설명으로 변경해봤습니다.(피드백 환영합니다)
  • megaphone.tsx 파일에서 테스트용 temp로 확성기 내용 개수 조절해서 테스트 해보실 수 있습니다.
  • (66번 줄 for문 조작하시면 됩니다. 넘어가는 시간은 78번 줄의 useInterval 값 조정해주시면 됩니다.)
  • 주석과 temp는 리뷰 끝나면 merge하기 전에 다 지우겠습니다

✅ 변경로직

@PHJoon PHJoon self-assigned this Aug 31, 2023
@PHJoon PHJoon changed the title [GGFE-225] 확성기 개수 제한 없이 작동하는 로직으로 변경 [GGFE-225] 확성기 개수 제한 없이 작동하게 css 변경 Aug 31, 2023

const megaphoneContent: IMegaphoneContent = {
megaphoneId: 2,
content: '오늘 하루 42GG는 내가 접수한다 📢😎',
Copy link
Member

Choose a reason for hiding this comment

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

등록된 확성기가 없을 때 content엔 '확성기: 오늘 42GG는 내가 접수한다 📢😎', intraId엔 '절찬 판매 중!' 이런 식으로 하는 건 어떨까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

좋네요! 바꾸겠습니다!

Copy link
Member

@hyobb109 hyobb109 left a comment

Choose a reason for hiding this comment

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

확성기 개수 많아도 애니메이션 잘 보이고 좋습니다! 고생 많으셨어요👍

Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

style 속성으로 넣으니까 확실히 한번에 알아보긴 좋네요 ㅋㅋㅋㅋ
확성기 애니메이션도 여전히 잘 동작하고 빈 배열일 경우에 아이템 소개 보여주는 것도 좋은 것 같아요!


확성기 내용이 많아지는 경우에(기본적으로 설정해주신 temp에서도 확인할 수 있습니닷) transform에서 오차가 누적되는지 (정확한 원인은 모르겠습니다 ^_ㅜ) 점점 아래 아이템으로 내려갈수록 살짝 덜그럭(?) 거리네요 😂 덜그럭거리는건 사실 보기엔 이상하지 않아서 큰 문제는 아닐 것 같긴 한데 개발자 도구로 봤을 때 확성기 아래에 div태그가 엄청 많이 달려있는게 괜찮은가? 하는 생각도 들어서 이 부분은 개선할 수 있는 방법이 있을지 좀 고민을 해 봐야 할 것 같아요 ㅋㅋㅋㅜ

근데 처음에 말했듯이 애초에 확성기 등록이 이렇게 많이 되는 경우가 없을 것 같아서 위 내용은 우선순위가 좀 떨어질 것 같긴 합니당 ㅎㅎ 수고하셨습니다!! 😆👍

@PHJoon
Copy link
Contributor Author

PHJoon commented Sep 5, 2023

확성기가 많은 경우에 마지막에 가까워질수록 덜그덕거리는거 같네요 😭 일단 작동하는데에는 문제 없어서 개선방법은 추후에 생각해보겠습니다!

@yoouyeon yoouyeon merged commit c83235f into main Sep 5, 2023
1 check passed
@hyobb109 hyobb109 deleted the GGFE-225-확성기-개수-제한-로직-변경 branch October 30, 2023 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants