Skip to content

[Application] Add Qt Designer MainWindow draft#121

Closed
muzygosu wants to merge 1 commit into
mainfrom
junyong
Closed

[Application] Add Qt Designer MainWindow draft#121
muzygosu wants to merge 1 commit into
mainfrom
junyong

Conversation

@muzygosu

Copy link
Copy Markdown
Collaborator

backup

Summary

  • Qt Designer 기반 MainWindow UI 초안을 추가했습니다.
  • 애플리케이션 UI에서 사용할 SVG 아이콘 리소스를 추가했습니다.
  • resources.qrc를 추가하고 safecrowd_app CMake 소스에 포함했습니다.

Related Issue

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
  • ctest --preset test-debug
  • Not run (reason below)

Risks / Follow-up

  • Qt Designer UI 초안만 추가한 상태이며, 기존 MainWindow.cpp 로직과의 실제 연결은 후속 작업에서 진행해야 합니다.
  • 로컬에서 Qt 앱 빌드 및 실행 검증은 별도로 필요합니다.

@muzygosu muzygosu requested a review from learncold as a code owner April 10, 2026 15:14
@learncold

Copy link
Copy Markdown
Owner

현재 변경은 merge하지 않고 종료하는 쪽으로 정리합니다.

정리 이유:

  • 이 PR의 mainwindow.ui 초안은 현재 application source-of-truth인 workspace shell 방향(Project / Authoring / Run / Analysis)과 맞지 않습니다.

esources.qrc / .ui는 실제 safecrowd_app에 연결되지 않아, 현재 상태로는 shipped UI를 바꾸지 못합니다.

  • 결과적으로 이 PR은 동작 변경 없이 설계 드리프트와 dead asset만 남기게 됩니다.

또한 기존 범위의 근거였던 #19는 더 구체적인 설계 반영 이슈들(#120, #80, #96)로 대체되어 이미 종료되었습니다.

따라서 이 PR은 close하고, 필요한 작업은 최신 main 기준에서 설계 반영 이슈를 직접 대상으로 새 PR로 다시 진행하는 것이 맞습니다.

@learncold learncold closed this Apr 13, 2026
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.

Task-Implement minimal run pause reset controls

2 participants