Skip to content

[Application] Improve door color visibility#187

Merged
learncold merged 1 commit into
mainfrom
codex/improve-door-color-visibility
May 6, 2026
Merged

[Application] Improve door color visibility#187
learncold merged 1 commit into
mainfrom
codex/improve-door-color-visibility

Conversation

@muzygosu
Copy link
Copy Markdown
Collaborator

@muzygosu muzygosu commented May 5, 2026

Summary

  • 문(Doorway) 연결선이 기존 파란색 계열에서는 배경 및 다른 연결 요소와 구분이 약해, 파란색의 보색인 주황색으로 변경해 대비와 가시성을 높인다.
  • 문 도구 버튼과 좌측 레이아웃 트리의 문 아이콘 색상도 같은 주황색으로 맞춘다.
  • 출구 색상은 기존 녹색 계열을 유지한다.

Related Issue

  • None (application-only PR)

Area

  • Engine
  • Domain
  • Application
  • Docs
  • Build
  • Analysis
  • Chore

Architecture Check

  • I kept the dependency direction application -> domain -> engine.
  • I did not add Qt UI code to src/domain.
  • I did not add domain or application dependencies to src/engine.
  • I used src/ as the include root.

Verification

  • cmake --preset windows-debug
  • cmake --build --preset build-debug --target safecrowd_app
  • ctest --preset test-debug
  • Not run (reason below)

Risks / Follow-up

  • application-only 시각 표시 변경이라 전체 CTest는 실행하지 않았다.
  • 가운데 Demo 문이 실제 Doorway로 분류되는 수정은 별도 Domain PR에서 처리한다.

@muzygosu muzygosu requested a review from learncold as a code owner May 5, 2026 09:33
Copy link
Copy Markdown
Owner

@learncold learncold left a comment

Choose a reason for hiding this comment

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

Reviewed locally. No blocking issues found. Verified with cmake --preset windows-debug, cmake --build --preset build-debug, and ctest --preset test-debug.

@learncold learncold merged commit cd54baa into main May 6, 2026
2 checks passed
@learncold learncold deleted the codex/improve-door-color-visibility branch May 6, 2026 10: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.

2 participants