Repository navigation
Develop - #108
Merged
Merged
Develop#108
Conversation
- .route-view[data-route=chat]의 min-height: 660px 제거 → 짧은 화면에서 채팅 레이아웃이 뷰포트를 넘어가 하단 목록·스크롤바가 잘리던 원인 - 100vh → 100dvh (모바일 브라우저 UI 대응) - .chat-layout에 align-items: stretch 추가 → 공유 규칙(.detail-layout 등)의 align-items: start가 상속돼 그리드 아이템이 셀 높이 대신 내용 높이로 부풀던 문제 차단 (이게 없으면 목록이 내부 스크롤을 못 가짐) - .conversation-list / [data-room-list]를 flex 컬럼으로 전환하고 스크롤 컨테이너에 flex:1 + min-height:0 적용 (내부 스크롤 보장) - scrollbar-gutter: stable + ::-webkit-scrollbar 스타일로 스크롤바 항상 노출 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- 하트비트가 없으면 유휴 WebSocket이 프록시(Render 등)에 의해 조용히 끊기고, 끊긴 사이 브로드캐스트된 메시지(거래 요청 카드 등)가 유실됨 - 서버: enableSimpleBroker에 setHeartbeatValue(10s/10s) + 전용 TaskScheduler - 클라이언트: heartbeatIncoming/Outgoing 10s 명시, reconnectDelay 3s Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- 심플 브로커는 미접속 클라이언트 메시지를 큐잉하지 않으므로, 재연결 직후 서버 상태 기준으로 히스토리와 거래 플로우를 다시 맞춤 - connectChatRoom onConnect에서 첫 연결은 건너뛰고 재연결부터 catchUpAfterReconnect() 실행 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- 저장은 호출자 트랜잭션 안에서, convertAndSend는 afterCommit 동기화로 이동 - 커밋 전 발행 시 롤백되면 유령 메시지가 생기고, 구독자가 아직 DB에 없는 행을 참조할 수 있었음 - 트랜잭션이 없는 호출 경로는 즉시 발행(기존 동작 유지) Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…ime-heartbeat fix: 채팅 거래 요청이 실시간으로 전달되지 않던 문제
…hat-list-scroll fix: 채팅방 목록이 많을 때 좌측 목록 스크롤 안 되던 문제
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.