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-146] 확성기 스타일 변경(흐르는 배너 -> 상하 슬라이드) #934

Merged
merged 6 commits into from
Aug 17, 2023

Conversation

PHJoon
Copy link
Contributor

@PHJoon PHJoon commented Aug 16, 2023

📌 개요

  • 확성기 스타일 변경(흐르는 배너 -> 상하 슬라이들)

💻 작업사항

  • 글자 수에 따라 흐르는 속도 조절하는 것보다 슬라이드 형식으로 보여주는 것이 더 나은 것 같아서 변경했습니다.
  • 무한 슬라이드는 구현하지 못했고 마지막에 도달하면 다시 첫번째 아이템까지 슬라이드가 거슬러 올라갑니다.
  • (시간 남으면 구현해보겠습니다.)
  • 기존 확성기 코드는 주석처리 해놨습니다.

✅ 변경로직

@PHJoon PHJoon self-assigned this Aug 16, 2023
Copy link
Contributor

@parksangmin1543 parksangmin1543 left a comment

Choose a reason for hiding this comment

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

저는 상하슬라이드형식이 이쁜것같아요. 검은색이 너무 가로지르는 느낌이라 30자 제한이니까 검색창정도 크기에 끝이 둥근형태도 나쁘지 않을것같습니다!

@PHJoon
Copy link
Contributor Author

PHJoon commented Aug 16, 2023

검은색 흐리게 + 글자 빛나게 + 노란색 연하게도 추가해놓겠습니다

<MegaphoneContainer clickPause={clickPause} play={play}>
{contents.map((content, idx) =>
<MegaphoneContainer
clickPause={clickPause}
Copy link
Member

Choose a reason for hiding this comment

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

clickPause가 작동을 안하는 것 같아요

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
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.

상하 슬라이드로 하는 편이 확성기 문구를 한눈에 보기도 좋은 것 같네요!
무한 슬라이드가 구현된다면 좋겠지만 만약에 안되더라도 다시 1번으로 거슬러 올라가는것도 저는 괜찮아보입니당
수고하셨어요!! 😆👍

Comment on lines +50 to +52
&.slideNext9 {
@include slide(-900%);
}
Copy link
Member

Choose a reason for hiding this comment

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

혹시 하루에 등록할 수 있는 확성기의 개수 제한이 있었나요?? 개수 관련해서 얘기가 나왔던 것 같긴 한데 어디에 정리해놨는지;; 기억이 안나네요...
만약에 제한이 있다면 이런 식으로 클래스를 나누어서 slide 효과를 줘도 될 것 같은데, 만약에 제한이 없다면 이렇게 클래스를 지정해두는 방법에는 한계가 있을 것 같아서 slide mixin 코드를 바로 style 속성으로 전달해서 translateY($num); 을 바로 지정해줘야 할 것 같아요.
근데 이렇게 했을 때 translateY($num);의 num의 절댓값이 꽤 커져도 괜찮은지는 또 잘 모르겠네요... 😵‍💫 저도 찾아보겠습니다...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저도 확실하진 않은데 최대 10개인걸로 기억하고 있습니다!

@PHJoon PHJoon merged commit ebc068b into main Aug 17, 2023
@PHJoon PHJoon deleted the GGFE-146-확성기-흐르는-속도-디자인 branch August 17, 2023 05:16
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.

4 participants