Skip to content

fix: 터미널 추가 영역 양옆 경계 표시 - #35

Merged
safethecode merged 1 commit into
mainfrom
fix/terminal-add-borders
Oct 19, 2026
Merged

safethecode merged 1 commit into
mainfrom
fix/terminal-add-borders

Conversation

@safethecode

Copy link
Copy Markdown
Owner

세로 + 영역의 양옆 구분이 분명해야 한다는 사용자 요청에 따라 전체 높이의 1pt 경계를 추가했습니다. 라이트 B9BDC6 / 다크555B67로 표시하며, 48pt 폭과32pt 버튼은 유지합니다. 파일 트리를 숨겨도 양옆 선이 남습니다.

검증: ./scripts/check.sh 전체36개 통과. 실제 별도 AppKit 앱에서 다크 탭/트리 숨김, 라이트 격자/트리 표시, native 축소와 plus 클릭을 확인했습니다. 디자인 게이트 PASS.

셀프 리뷰: 내부 경계라 canvas/파일 트리 폭과 버튼 클릭 영역을 바꾸지 않습니다. 터미널 본문 포커스 테두리는 그대로입니다. 오래된 기본4칸 최소 폭 문서를 현재890pt로 고쳤습니다. 관찰 기록.

@coderabbitai

coderabbitai Bot commented Oct 19, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 52 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 0b3a6e89-6364-4f25-b169-09af21d7850d
📥 Commits

Reviewing files that changed from the base of the PR and between 4a1689b and 833f7d1.

📒 Files selected for processing: 8
  • DESIGN.md
  • Sources/Chop/AppearanceMode.swift
  • Sources/Chop/TerminalAreaView.swift
  • design/evidence/terminal-add-borders.md
  • design/failure-events.jsonl
  • design/gate-contract.json
  • design/gate-report.json
  • design/review.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@safethecode
safethecode merged commit aa2af72 into main Oct 19, 2026
2 checks passed
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.

1 participant