Skip to content

fix: scope inactive ticker modes - #2046

Merged
crossle merged 2 commits into
mainfrom
fix/scoped-ticker-mode
Sep 3, 2026
Merged

fix: scope inactive ticker modes#2046
crossle merged 2 commits into
mainfrom
fix/scoped-ticker-mode

Conversation

@YeungKC

@YeungKC YeungKC commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • remove the global inactive TickerMode so chat scrolling keeps animating when the desktop window loses focus
  • add AppActiveTickerMode and apply it to the input subtree, image preview, and message item
  • keep VoiceRecorderBottomBar outside the input ticker scope
  • add an inactive/resumed regression test covering message ticker state and sibling scroll animation

Tests

  • flutter analyze --no-pub lib/app.dart lib/ui/home/chat/input_container.dart lib/utils/app_lifecycle.dart lib/widgets/message/item/image/image_preview_page.dart lib/widgets/message/message.dart test/widgets/message/message_context_test.dart
  • flutter test --no-pub test/widgets/message/message_context_test.dart --plain-name "inactive mutes message ticker while sibling scroll still animates"
  • flutter test --no-pub test/ui/home/chat/voice_recorder_overlay_test.dart
  • flutter test --no-pub test/widgets/message/image_preview_page_test.dart

@YeungKC
YeungKC requested a review from boyan01 September 3, 2026 05:45
@YeungKC
YeungKC marked this pull request as ready for review September 3, 2026 05:45
@YeungKC
YeungKC marked this pull request as draft September 3, 2026 05:55
@YeungKC
YeungKC marked this pull request as ready for review September 3, 2026 05:56
@crossle
crossle merged commit 65c8004 into main Sep 3, 2026
3 checks passed
@crossle
crossle deleted the fix/scoped-ticker-mode branch September 3, 2026 06:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants