[CLX-3720][S] Add a11y at Ai chatbot

[Ahmed-Naguib93]

What is new?

Add a11y for

1. Chatbot
2. Quizes
3. FlashCards

refs: CLX-3720
builds: Student
affects: Student
release note: none

test plan: none

[claude]

Claude Code Review

Updated: 2025-12-26

• Potential null crash in AssistQuizViewModel.isCorrect(): Signature changed to accept AnswerOption? but lacks proper nil handling in error cases. The guard statement could mask silent failures when answer becomes nil unexpectedly.

• Logic error in AssistFeedbackView feedback dismissal: The feedback message will never be seen because opacity is set to 0 immediately, then animated but the view is hidden before animation completes. The timing logic (3.0s delay) doesn't match the animation duration.

• Type signature inconsistency in AssistFeedbackView: Changed from AssistChatMessageViewModel.OnFeedbackChange to inline (Bool?) -> Void. While compatible, this removes the abstraction and makes refactoring harder if the type changes.

• Missing accessibility announcement context in AssistChatView: VoiceOver focus is set on new messages, but message content isn't automatically announced when focused. Consider adding accessibilityValue or accessibilityHint.

• String formatting typo in Localizable.xcstrings: Entry 'Your answer wrong.' should be 'Your answer is wrong.' for grammatical correctness in accessibility announcements.

✅ Approved

[inst-danger]

	Warnings
⚠️	This pull request will not generate a release note.

Affected Apps: Student

Builds: Student

Coverage	New %	Master %	Delta
Canvas iOS	91.23%	81.06%	10.16%

Generated by 🚫 dangerJS against f419bbc

[inst-danger]

Builds

Commit: Merge branch 'master' into feature/career-add-a11y-chatbot-CLX-3720 (f419bbc)
Build Number: 1095
Built At: Jan 06 13:46 CET (01/06 05:46 AM MST)

Student

The base branch was changed.