Skip to content
This repository was archived by the owner on Jul 15, 2026. It is now read-only.

Add gesture inversion settings to Apple Watch app - #11

Merged
donny-son merged 7 commits into
mainfrom
claude/apple-watch-timer-settings-3GCVW
Mar 1, 2026
Merged

Add gesture inversion settings to Apple Watch app#11
donny-son merged 7 commits into
mainfrom
claude/apple-watch-timer-settings-3GCVW

Conversation

@donny-son

Copy link
Copy Markdown
Contributor
  • Center timer in the middle row with spacers
  • Add settings gear button to the right of the timer
  • Create SettingsView with gesture inversion toggle
  • Add isInverted property to GestureManager with UserDefaults persistence
  • In inverted mode: counterclockwise → next, clockwise → previous

https://claude.ai/code/session_01NwfWjGZ6sYHeWY1c63VyUr

- Center timer in the middle row with spacers
- Add settings gear button to the right of the timer
- Create SettingsView with gesture inversion toggle
- Add isInverted property to GestureManager with UserDefaults persistence
- In inverted mode: counterclockwise → next, clockwise → previous

https://claude.ai/code/session_01NwfWjGZ6sYHeWY1c63VyUr

@donny-son donny-son left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Code Review 🥟

깔끔한 PR입니다! 제스처 반전 설정 기능이 잘 구현되었네요.

✅ 잘된 점

  • UserDefaults 영속화 적절히 처리됨
  • (rotationX > 0) != isInverted 로직이 간결함
  • SwiftUI 상태 관리 패턴 잘 따름
  • Preview 포함된 점 👍

💡 개선 제안 (선택사항)

1. @AppStorage 사용 고려

// Before
@Published var isInverted: Bool {
    didSet { UserDefaults.standard.set(isInverted, forKey: "gestureInverted") }
}

// After - more SwiftUI idiomatic
@AppStorage("gestureInverted") var isInverted = false

단, ObservableObject에서 @AppStorage@published 조합이 까다로울 수 있어서 현재 방식도 충분히 괜찮습니다.

2. Settings 진입점 피드백
설정 버튼 탭 시 햅틱 피드백 추가하면 UX 향상될 수 있음:

WKInterfaceDevice.current().play(.click)

전반적으로 머지해도 될 것 같습니다! 🚀

@donny-son
donny-son merged commit b084299 into main Mar 1, 2026
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants