Skip to content

feat: add haptic feedback to additional sheet presentations#124

Merged
Riochin merged 3 commits intomainfrom
claude/codebase-structure-review-011CUvVxLzycqiywMs7QRMcC
Nov 8, 2025
Merged

feat: add haptic feedback to additional sheet presentations#124
Riochin merged 3 commits intomainfrom
claude/codebase-structure-review-011CUvVxLzycqiywMs7QRMcC

Conversation

@RedBlueBird
Copy link
Collaborator

Overview

Extend haptic feedback system to provide tactile responses for more sheet and modal presentations throughout the app.

Changes:

  • UserProfileBottomSheetView: Add light impact on open/close in NationalRankingView, TeamRankingView, and GroupDetailView
  • BadgeDetailView: Add light impact on open/close for all three badge view types (BadgeItemView, BadgeCardButton, Simple3DBadgeView)
  • MissionSection: Add light impact when opening/closing mission detail sheet
  • GroupDetailView: Add light impact when opening/closing group detail sheet

All feedback follows the same pattern: light impact on open and light impact on close, providing consistent tactile response across the app.

Related Issue

Changes

How to Test

Build & Run

Screenshots/Videos

Points for Review

Extend haptic feedback system to provide tactile responses for
more sheet and modal presentations throughout the app.

Changes:
- UserProfileBottomSheetView: Add light impact on open/close in
  NationalRankingView, TeamRankingView, and GroupDetailView
- BadgeDetailView: Add light impact on open/close for all three
  badge view types (BadgeItemView, BadgeCardButton, Simple3DBadgeView)
- MissionSection: Add light impact when opening/closing mission detail sheet
- GroupDetailView: Add light impact when opening/closing group detail sheet

All feedback follows the same pattern: light impact on open and light
impact on close, providing consistent tactile response across the app.
Add tactile feedback when users flip the badge to see the back side
in BadgeDetailView's RotatableBadgeView component.

Changes:
- RotatableBadgeView: Medium impact haptic when badge is flipped,
  providing stronger feedback for this more significant 3D interaction
Fix build errors where IdentifiableUUID needs to conform to Equatable
to work with SwiftUI's onChange(of:) modifier.

Changes:
- NationalRankingView: Add Equatable conformance to IdentifiableUUID
- TeamRankingView: Add Equatable conformance to IdentifiableUUID

This fixes the compilation errors:
"referencing instance method 'onChange(of:initial:_:)' on 'Optional'
requires that 'IdentifiableUUID' conform to 'Equatable'"
Copy link
Contributor

@Riochin Riochin left a comment

Choose a reason for hiding this comment

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

FLVGTM
(feedback looks very good to me)

User profile as well

@Riochin Riochin merged commit 1f97347 into main Nov 8, 2025
1 check 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.

3 participants