[go: nahoru, domu]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix staggered animation page #10809

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

atsansone
Copy link
Contributor
@atsansone atsansone commented Jun 26, 2024
  • Pulls out the second, complex staggered animation example from the note into its own section. This was done in lieu of creating a video or animation. This includes 3 new screenshots based on the Flutter app in the example. Fixes Add staggered_pic_selection video to 'Staggered animations' page #5792
  • Fixes an omission in the figure caption code to work with different sizes.
  • Reorganizes the page to provide more context before starting the procedure / example.

@flutter-website-bot
Copy link
Collaborator
flutter-website-bot commented Jun 26, 2024

Visit the preview URL for this PR (updated for commit 69df676):

https://flutter-docs-prod--pr10809-fix-5792-6lfeljrw.web.app

@sfshaza2
Copy link
Contributor

PLEASE, add a sentence in addition to a bug link. It's a much more efficient use of the reviewer's time.

src/content/ui/animations/staggered-animations.md Outdated Show resolved Hide resolved

When the controlling animation's value changes,
the new animation's value changes, triggering the UI to update.

### Tween a widget's width
Copy link
Contributor

Choose a reason for hiding this comment

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

Rephrase; Tween is not a verb.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In the animation business, it is: https://en.wikipedia.org/wiki/Inbetweening.

Copy link
Member

Choose a reason for hiding this comment

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

Perhaps "Interpolate" or "Transition" would read better when a verb is needed? That would avoid conflation with "tween" as the type (or instance of that type), and potentially allow better machine translation too.

"Interpolate between a widget's widths" or "Transition between a widget's widths"

Copy link
Contributor
@sfshaza2 sfshaza2 Jun 27, 2024

Choose a reason for hiding this comment

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

Yes, to either of Parker's suggestions.


To review other available pre-defined animation curves, consult [`Curves`][].

### Tween a widget's border radius
Copy link
Contributor

Choose a reason for hiding this comment

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

Again, don't use "Tween" as a verb.

The following code builds the tween for the `borderRadius` property
(which controls the roundness of the square's corners),
using `BorderRadius.circular()`.

```dart
```dart title="Tweening the widget's corners (main.dart)"
Copy link
Contributor

Choose a reason for hiding this comment

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

Ditto.

@atsansone
Copy link
Contributor Author

PLEASE, add a sentence in addition to a bug link. It's a much more efficient use of the reviewer's time.

Agreed and fixed.

Copy link
Member
@parlough parlough left a comment

Choose a reason for hiding this comment

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

I haven't reviewed this whole PR, but have two drive by comments:


```dart
```dart title="Tweening the widget's width (main.dart)"
Copy link
Member

Choose a reason for hiding this comment

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

The titles on code blocks aren't really meant for descriptions but rather filenames. Is the title necessary here?

Copy link
Contributor

Choose a reason for hiding this comment

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

I wondered about that.


When the controlling animation's value changes,
the new animation's value changes, triggering the UI to update.

### Tween a widget's width
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps "Interpolate" or "Transition" would read better when a verb is needed? That would avoid conflation with "tween" as the type (or instance of that type), and potentially allow better machine translation too.

"Interpolate between a widget's widths" or "Transition between a widget's widths"

@atsansone atsansone requested a review from jmagman as a code owner June 27, 2024 20:26
atsansone and others added 3 commits June 27, 2024 15:26
Co-authored-by: Shams Zakhour (ignore Sfshaza) <44418985+sfshaza2@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review.tech Awaiting Technical Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add staggered_pic_selection video to 'Staggered animations' page
5 participants