Skip to content

fix: 화면 배경 블러 제거 및 풀이 수 숫자 입력 제한 - #25

Merged
LATE-BL00MER merged 1 commit into
developfrom
fix/opaque-surfaces-numeric-history-limit
Oct 15, 2026
Merged

LATE-BL00MER merged 1 commit into
developfrom
fix/opaque-surfaces-numeric-history-limit

Conversation

@LATE-BL00MER

Copy link
Copy Markdown
Contributor

관련 이슈

해당 없음.

변경사항

설정과 풀이 기록 등 공통 화면에서 창 뒤의 색상이 비쳐 글을 읽기 어려웠고, 최근 풀이 개수 입력란에 문자열을 입력할 수 있었습니다.

  • 공통 배경을 다크·라이트 모드에 대응하는 불투명 시스템 색상으로 바꾸고, 설정 창과 제목 표시줄에서도 배경 블러를 제거합니다.
  • 풀이 개수 입력란은 AppKit 편집 검증으로 숫자(0–9)만 허용합니다. 문자열 입력과 문자가 섞인 붙여넣기를 거부하며, 기존 1–100 범위 보정과 빈 입력 복구를 유지합니다.

변경·영향 범위

담당 주 영역: 앱 UI — 공통 배경 및 설정의 최근 풀이 입력란.

  • 앱 기능 (Capture / AI / Storage / UI)
  • StoreKit·사용량 정책
  • API·프록시
  • 앱 설정·권한·릴리스
  • 테스트·CI·개발 도구
  • 규칙·문서 (AGENTS.md, docs/)
  • 기타:

PR 분류

  • 일반 변경
  • Critical Path 변경

사전 예측

  • 예상 소요·기대 효과: 화면 가독성을 개선하고 잘못된 풀이 개수 입력을 편집 단계에서 차단합니다.
  • 예상 병목·리스크: SwiftUI 입력 변환만으로는 편집 중 문자열을 차단하지 못할 수 있어 AppKit 검증을 사용합니다.
  • 담당 영역 밖 확인 사항: 공통 배경을 사용하는 결과·기록 화면에도 불투명 배경이 적용됩니다.

실제 결과·트러블슈팅

  • 실제 소요·반영 결과: UI 파일 두 개만 수정했습니다.
  • 실제 병목·해결 과정: 네이티브 NSTextField와 Formatter로 직접 입력과 붙여넣기에 같은 검증을 적용했습니다.
  • 배운 점·후속 작업: 공통 배경과 설정 창의 기본 제목 표시줄 배경을 함께 처리했습니다.

검증 결과

  • macOS xcodebuild ... CODE_SIGNING_ALLOWED=NO test: 테스트 72개 통과. 서버 변경을 제외한 별도 소스 사본에서 실행했으며, 이번 커밋의 두 UI 파일과 동일함을 확인했습니다.
  • Release 아카이브 성공 및 번들 내 불필요한 소스·문서 파일 없음 확인.
  • xcrun swift-format lint --strict Paxo/UI/PaxoStyle.swift Paxo/UI/SettingsView.swift, git diff --check: 통과.
  • 수동 확인: 다크·라이트 배경, 문자 입력과 혼합 붙여넣기 차단, 숫자 붙여넣기, 빈 값 복구와 1–100 보정 확인.
  • API Contract 앱·프록시 양쪽 반영 여부: 해당 없음. 서버·로그인·구독 로직 변경 없음.
  • 시크릿 포함 여부: 없음. 커밋에 UI 파일 두 개만 포함되는 것을 확인했습니다.

@github-actions

Copy link
Copy Markdown

AI 사전 코드 리뷰 (파일럿)

검토 SHA: 34ccd3d75922 · 방식: 전체
상태: 미해결 0 · 해결 0 · 오탐 0 · 위험 수용 0
AI 호출: 1/20

설정 창의 유리가 제거되고 불투명 배경(WindowBackground) 스타일로 변경되었으며, 메뉴에 표시할 풀이 수 입력란에 숫자만 허용하도록 NSViewRepresentable 기반의 HistoryLimitField 및 Formatter가 도입되었습니다. 코드 컨벤션과 안전성 규칙을 준수하고 있습니다.

ID 심각도 확신·구분 위치 문제 상태
— — — — 수정할 가치가 높은 문제를 찾지 못했습니다. —

테스트 공백

  • HistoryLimitFormatter의 숫자 필터링 검증(문자 입력 및 붙여넣기 차단)에 대한 단위 테스트가 추가되면 더욱 안정적입니다.

사람의 판정: /ai-review false-positive AX-... 이유(10자 이상), /ai-review accepted-risk AX-... 이유(10자 이상), /ai-review reopen AX-...
AI 결과는 승인이나 머지 조건을 대체하지 않습니다.

@LATE-BL00MER
LATE-BL00MER merged commit 50f1e7d into develop Oct 15, 2026
7 checks passed
@LATE-BL00MER
LATE-BL00MER deleted the fix/opaque-surfaces-numeric-history-limit branch October 15, 2026 04:57
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