Skip to content

1079 update animation docs include new v12 changes to the control visibility logic - #1129

Open
luke-whos-here wants to merge 4 commits into
mainfrom
1079-update-animation-docs-include-new-v12-changes-to-the-control-visibility-logic
Open

1079 update animation docs include new v12 changes to the control visibility logic#1129
luke-whos-here wants to merge 4 commits into
mainfrom
1079-update-animation-docs-include-new-v12-changes-to-the-control-visibility-logic

Conversation

@luke-whos-here

Copy link
Copy Markdown
Contributor

This PR updates various pages to document changes to keyframe animations introduced in AvaloniaUI/Avalonia#20820

  • Main page on keyframe animations includes a new section on playback behavior.
  • Reference page on animation settings includes a new table listing playback behavior options.
  • Associated pages that touch on the topic have notes and/or crosslinks added to improve discoverability.
  • Control transitions page has a note to specify that transitions are not affected.
  • Various opportunistic style edits.
  • Upgrade some samples to use XAML preview.

- Document playback behavior on keyframe-animations page.
- Document attribute options on animation-settings page.
- Opportunistic style edits, including removal of redundant tables copied on both pages.
- Edit animations page to mention playback behavior.
- Opportunistic upgrade of triggering demo to use XAML live preview.
Add notes to pages peripherally touching the topic of keyframe animations pausing when hidden, and add crosslinks to the main keyframe-animations page to improve discoverability.

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI 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.

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 8 comments.

Comment thread docs/graphics-animation/animations.md Outdated
Comment thread docs/graphics-animation/effects.md Outdated
Comment thread docs/graphics-animation/effects.md Outdated
Comment thread docs/graphics-animation/control-transitions.md Outdated
Comment thread docs/graphics-animation/animation-settings.md Outdated
Comment thread docs/graphics-animation/keyframe-animations.md Outdated
Comment thread docs/graphics-animation/keyframe-animations.md Outdated
Comment thread docs/app-development/performance.md Outdated
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.

Update animation docs: include new v12 changes to the control visibility logic

2 participants