🎨 Palette: Add non-visual feedback to diagnostic report copy#1795
🎨 Palette: Add non-visual feedback to diagnostic report copy#1795emkey1 wants to merge 1 commit into
Conversation
Added `withAnimation` wrapper to "Copy Report" state changes for smoother UI transitions. Introduced haptic feedback (`UINotificationFeedbackGenerator`) and VoiceOver announcements (`UIAccessibility.post`) to provide vital non-visual confirmation when a user triggers the transient copy action. Also added explicit accessibility hint. Co-authored-by: emkey1 <18100932+emkey1@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
💡 What:
Added haptic feedback and VoiceOver announcement when copying the diagnostic report in
AppDiagnosticsView.swift. UsedwithAnimationfor the transient button state change and added explicitaccessibilityHint.🎯 Why:
Copying text in transient contexts offers no non-visual confirmation by default. This makes it unclear for VoiceOver users or those relying on haptics if the action succeeded. The animation prevents the visual UI from feeling jumpy.
♿ Accessibility:
Users will now feel a haptic pulse and hear VoiceOver read "Copied report" when interacting with the button. Added
.accessibilityHintexplaining the button's action.PR created automatically by Jules for task 17174169958827067340 started by @emkey1