🎨 Palette: AppDiagnosticsView Micro-UX & Accessibility Improvements#1775
🎨 Palette: AppDiagnosticsView Micro-UX & Accessibility Improvements#1775emkey1 wants to merge 3 commits into
Conversation
…iew buttons 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. |
…iew buttons Co-authored-by: emkey1 <18100932+emkey1@users.noreply.github.com>
…iew buttons Co-authored-by: emkey1 <18100932+emkey1@users.noreply.github.com>
💡 What: Added
withAnimationblocks around the state changes for the "Copied Report" transient text, and added explicitaccessibilityLabelandaccessibilityHintmodifiers to the "Copy Report" and "Run Diagnostics" buttons inAppDiagnosticsView.🎯 Why: Previously, the "Copied!" text would snap in and out abruptly, creating a jarring visual experience. Wrapping the state toggle in
withAnimationprovides a smooth cross-dissolve transition. Additionally, VoiceOver users lacked clear context for what the "Copy Report" and "Run Diagnostics" buttons did, especially regarding the transient success state. The new accessibility modifiers ensure the actions and their outcomes are clearly announced.♿ Accessibility:
.accessibilityLabelto dynamically read "Copied Report" when the copy action succeeds..accessibilityHintto the copy button to clarify it copies the diagnostic report text to the clipboard..accessibilityHintto the run button to clarify it executes the app's diagnostic sanity checks.PR created automatically by Jules for task 7435828072089591663 started by @emkey1