Skip to content

[two_dimensional_scrollables] Fix TreeView bug when animation duration is zero - #7475

Merged
auto-submit[bot] merged 8 commits into
flutter:mainfrom
Mairramer:fix/treesliver-unpack-with-no-animation
Sep 11, 2024
Merged

[two_dimensional_scrollables] Fix TreeView bug when animation duration is zero#7475
auto-submit[bot] merged 8 commits into
flutter:mainfrom
Mairramer:fix/treesliver-unpack-with-no-animation

Conversation

@Mairramer

@MairramerMairramer commented Aug 21, 2024

Copy link
Copy Markdown
Contributor

Fixes#153889 an issue where nodes were being removed incorrectly when using AnimationStyle.noAnimation or the animation duration was zero seconds, which previously caused the erratic behavior due to hidden state updates. Similar to flutter/flutter#153890.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
  • All existing and new tests are passing.
    If you need help, consider asking for advice on the #hackers-new channel on Discord.

@Mairramer
Mairramer requested a review from Piinks as a code ownerAugust 21, 2024 23:07
@PiinksPiinks changed the title Fix a bug when animation duration is zero[two_dimensional_scrollables] Fix TreeView bug when animation duration is zeroAug 22, 2024
@PiinksPiinks added the p: two_dimensional_scrollables Issues pertaining to the two_dimensional_scrollables package label Aug 22, 2024
Comment threadpackages/two_dimensional_scrollables/test/tree_view/tree_test.dart Outdated
@Mairramer
Mairramer requested a review from PiinksAugust 28, 2024 12:58

@PiinksPiinks 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.

We should have an issue specific to this. Can you check https://github.com/flutter/flutter/issues?q=is%3Aopen+is%3Aissue+label%3A%22p%3A+two_dimensional_scrollables%22 and see if one exists? The other issue is for the 1D version in the framework.

@Mairramer
Mairramer requested a review from PiinksAugust 30, 2024 11:37

@PiinksPiinks 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.

LGTM, thank you for the contribution!

Comment threadpackages/two_dimensional_scrollables/lib/src/tree_view/tree.dart Outdated

@QuncCcccccQuncCccccc 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.

LGTM! Thanks for the contribution:)

@QuncCcccccQuncCccccc added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 11, 2024
@auto-submit
auto-submitBot merged commit 91caa7a into flutter:mainSep 11, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 12, 2024
auto-submitBot pushed a commit to flutter/flutter that referenced this pull request Sep 12, 2024
flutter/packages@4c18648...91caa7a
2024-09-11 50643541+Mairramer@users.noreply.github.com [two_dimensional_scrollables] Fix TreeView bug when animation duration is zero (flutter/packages#7475)
2024-09-11 109111084+yaakovschectman@users.noreply.github.com [google_maps_flutter_platform_interface] Fix typo in CameraUpdateNewLatLngBounds.toJson (flutter/packages#7626)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…n is zero (flutter#7475)
Fixes [#153889 ](flutter/flutter#154292) an issue where nodes were being removed incorrectly when using `AnimationStyle.noAnimation `or the animation duration was zero seconds, which previously caused the erratic behavior due to hidden state updates. Similar to flutter/flutter#153890.
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
…n is zero (flutter#7475)
Fixes [#153889 ](flutter/flutter#154292) an issue where nodes were being removed incorrectly when using `AnimationStyle.noAnimation `or the animation duration was zero seconds, which previously caused the erratic behavior due to hidden state updates. Similar to flutter/flutter#153890.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit Appp: two_dimensional_scrollablesIssues pertaining to the two_dimensional_scrollables package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[two_dimensional_scrollables] - TreeViewNode collapse with animation duration zero has erratic behavior

3 participants

@Mairramer@Piinks@QuncCccccc