Skip to content

Snackbar timeout#173127

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

Snackbar timeout#173127
victorsanni wants to merge 5 commits intoflutter:masterfrom
victorsanni:snackbar-timeout

Conversation

@victorsanni
Copy link
Contributor

No description provided.

@github-actions github-actions bot added framework flutter/packages/flutter repository. See also f: labels. f: material design flutter/packages/flutter/material repository. d: api docs Issues with https://api.flutter.dev/ d: examples Sample code and demos labels Aug 1, 2025
Copy link
Contributor

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

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.

@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 f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant