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/chat date divider #478

Merged
merged 2 commits into from
Mar 27, 2024
Merged

feat/chat date divider #478

merged 2 commits into from
Mar 27, 2024

Conversation

yumincho
Copy link
Member

요약 *

It closes #471 .

스크린샷

image

이후 Task *

  • 디자인 작업 없이 구현하였기 때문에 추후 디자이너 분의 검토 및 Figma 반영이 필요합니다.

@yumincho yumincho added FE Tasks for the front-end part Chat labels Mar 20, 2024
@yumincho yumincho self-assigned this Mar 20, 2024
Copy link

netlify bot commented Mar 20, 2024

Deploy Preview for biseo-preview ready!

Name Link
🔨 Latest commit 0ee3b89
🔍 Latest deploy log https://app.netlify.com/sites/biseo-preview/deploys/65fac7d301b66700085dd0a9
😎 Deploy Preview https://deploy-preview-478--biseo-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@Sanghou Sanghou left a comment

Choose a reason for hiding this comment

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

  1. 좌우 정렬이 안 맞는 것 같아요!
  2. nowrap인데 혹시 화면 가로 폭을 더 줄이면 어떻게 되나요?

@yumincho
Copy link
Member Author

yumincho commented Mar 20, 2024

@Sanghou

  1. 스크롤 바 때문에 좌우 정렬이 안 맞게 보이는 것 같아요. 혹시 이걸 말씀하신 게 아니라면 사진 첨부해주시면 감사하겠습니다!
image
  1. 저희 지금 반응형이 없어서 가로 폭 줄여도 그대로일 거예요 .. 😂

@Sanghou
Copy link
Collaborator

Sanghou commented Mar 20, 2024

@Sanghou

  1. 스크롤 바 때문에 좌우 정렬이 안 맞게 보이는 것 같아요. 혹시 이걸 말씀하신 게 아니라면 사진 첨부해주시면 감사하겠습니다!
image 2. 저희 지금 반응형이 없어서 가로 폭 줄여도 그대로일 거예요 .. 😂

뭔가 전반적으로 스크롤바 width 때문에 좌측으로 치우친 느낌이었군요 😢
스크롤바가 해당 라인 옆에 있어도 스크롤 바 옆에 margin이나 padding으로 인해 조금 더 우측에 여백이 있는 느낌이긴하네요

image

개인적인 느낌일 수도 있어서 다른 사용자분들이 보셨을 때 어색한지만 확인되면 좋을 것 같습니다.
기능이나 구현은 너무 좋은 것 같습니다 👍👍


디자인 비교군은 Slack과 채널톡, 카카오톡으로 보았습니다.

  1. Slack은 좌우 끝까지 붙어있음.
image 2. 채널톡과 카카오톡은 따로 선은 없고 텍스트만 띄워둠 카카오는 스크롤바 범위를 포함해서 중앙 정렬 스크린샷 2024-03-20 오후 8 42 17 image

Copy link
Collaborator

@rjsdn0 rjsdn0 left a comment

Choose a reason for hiding this comment

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

lgtm

@yumincho yumincho merged commit 7032f83 into main Mar 27, 2024
5 checks passed
@yumincho yumincho deleted the feat/chat-date-divider branch March 27, 2024 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chat FE Tasks for the front-end part
Projects
None yet
Development

Successfully merging this pull request may close these issues.

채팅 로그에 날짜 표시
3 participants