Skip to content

Snackbar timeout - #173127

Closed
victorsanni wants to merge 5 commits into
flutter:masterfrom
victorsanni:snackbar-timeout
Closed

victorsanni wants to merge 5 commits into
flutter:masterfrom
victorsanni:snackbar-timeout

Conversation

@victorsanni

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos labels Aug 1, 2025

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request changes the behavior of SnackBar timeouts. Now, a SnackBar with an action will not time out, regardless of accessibility settings. This is a good improvement for user experience. The implementation correctly moves the timeout logic to ScaffoldMessengerState and updates documentation and tests accordingly.

However, I've found a potential accessibility regression where snackbar animations are no longer disabled when accessibility features are enabled. I've left a comment with details on how to address this.

Comment thread packages/flutter/lib/src/material/snack_bar.dart
@victorsanni
victorsanni marked this pull request as draft August 4, 2025 17:48
@victorsanni
victorsanni deleted the snackbar-timeout branch January 12, 2026 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos framework flutter/packages/flutter repository. See also f: labels. p: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant