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

[BUILD FAIL] [Feat/#299] 최적의 회의시간 뷰 인원수 Chip과 Tooltip 추가 및 방장 페이지 관련 수정 #316

Merged
merged 8 commits into from
Aug 20, 2024

Conversation

simeunseo
Copy link
Member

@simeunseo simeunseo commented Aug 17, 2024

🌀 해당 이슈 번호

🔹 어떤 것을 변경했나요?

  • 인원수 표시 Chip 구현
  • Chip hover시 tooltip 구현
  • 방장 페이지 관련 모든 css 변경사항 처리
    • 종합일정시간표 하단 dim 추가
    • 최적의회의시간/종합일정시간표 탭 이동시 색깔 수정
    • gap 수정

🔹 어떻게 구현했나요?

tooltip의 visibility를 hidden으로 해둔뒤, hover시 visible로 바뀌도록 구현했습니다.

🔹 PR 포인트를 알려주세요!

🔹 스크린샷을 남겨주세요!

2024-08-18.3.37.13.mov

@simeunseo simeunseo added the feat new feature label Aug 17, 2024
@simeunseo simeunseo requested a review from ljh0608 August 17, 2024 19:00
@simeunseo simeunseo self-assigned this Aug 17, 2024
@simeunseo simeunseo added the 은서 은서의 개발 라벨 label Aug 17, 2024
Copy link
Member

@ljh0608 ljh0608 left a comment

Choose a reason for hiding this comment

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

리뷰 확인 후 머지해주세요~

background-color: ${theme.colors.grey8};
align-self: flex-end;
margin-left: auto;
cursor: pointer;
Copy link
Member

Choose a reason for hiding this comment

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

p2)
해당 hover로 인해 유저가 클릭했을 때 어떤 동작이 있을거라고 예상할 수도 있을 것 같아요.
클릭했을시에 어떠한 동작이 있는것은 아니니 pointer는 제거하는게 어떨까요??

Copy link
Member Author

Choose a reason for hiding this comment

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

아 저는 호버 영역에도, 커서를 갖다대면 무언가 반응형 동작이 일어난다는 점을 인지시키기 위해서 cursor pointer를 주는 편인데요..! 이 부분은 ux적인 측면인거같은데 정답은 모르겠네요.. 호버 이후에 인해 무언가 추가동작을 유도하는 건 아니니 재훈님 말도 이해가 가긴하는데 디자인쪽에 한번 의견 구해보겠습니다!

Copy link
Member Author

Choose a reason for hiding this comment

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

태희님이 기본 커서로 제안해주셨습니다! 생각해본적 없던 부분인데 배워갑니당 👍

Copy link
Member

Choose a reason for hiding this comment

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

저도 항상 배웁니다 감사합니다!

@simeunseo simeunseo merged commit 0273659 into develop Aug 20, 2024
1 of 2 checks passed
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

빌드에 실패했습니다.

@github-actions github-actions bot changed the title [Feat/#299] 최적의 회의시간 뷰 인원수 Chip과 Tooltip 추가 및 방장 페이지 관련 수정 [BUILD FAIL] [Feat/#299] 최적의 회의시간 뷰 인원수 Chip과 Tooltip 추가 및 방장 페이지 관련 수정 Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat new feature size/XL 은서 은서의 개발 라벨
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 최적의 회의시간 뷰 인원수 추가 및 툴팁
2 participants