Skip to content

chore(AnimationContainer): fix animation container position #2822

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

Merged
merged 2 commits into from
Mar 12, 2025

Conversation

xristianstefanov
Copy link
Contributor

Comes from client: 1681523

@xristianstefanov xristianstefanov added the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Mar 12, 2025
@xristianstefanov xristianstefanov self-assigned this Mar 12, 2025
@xristianstefanov xristianstefanov requested review from a team as code owners March 12, 2025 11:24
Copy link
Contributor

Hello @xristianstefanov,

Check the below option if you would like to automatically generate PR to production. The automation uses the branch for the cherry-pick, and then will delete the branch. Please, do not delete it manually.

  • create PR to production

Co-authored-by: Dimo Dimov <961014+dimodi@users.noreply.github.com>
@xristianstefanov xristianstefanov merged commit 8454561 into master Mar 12, 2025
Copy link
Contributor

Here you are the link to your PR to production: 2825

@github-actions github-actions bot deleted the animation-container-fix-position branch March 12, 2025 12:00
Copy link
Contributor

@ikoevska ikoevska left a comment

Choose a reason for hiding this comment

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

Reviewed.

@@ -12,7 +12,7 @@ position: 0

The <a href="https://www.telerik.com/blazor-ui/animation-container" target="_blank">Blazor Animation Container component</a> enables you to create messages and popups or expandable containers. It lets you define its animation, size and position, and arbitrary content.

>tip If you are looking for an option to create [Notification](slug:notification-overview), [Tooltip](slug:tooltip-overview) or expandable container such as [Drawer](slug:drawer-overview), you may use the dedicated components.
>tip The `AnimationContainer` animates and renders content in-place and does not have all the features of a true popup. [Compare the abilities of all Telerik Blazor popup components](slug:common-kb-popup-component-comparison) to verify if the [Popup](slug:popup-overview) or [Popover](slug:popover-overview) components are more suitable for your needs. If you are looking for an option to create [Notification](slug:notification-overview), [Tooltip](slug:tooltip-overview) or expandable container such as [Drawer](slug:drawer-overview), you may use the dedicated components.
Copy link
Contributor

Choose a reason for hiding this comment

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

Note that

"Can" is used to express ability or capability, while "may" is used to express permission or possibility

(Source)

In this case, I would recommend going with you can use the respective components.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-to-production Use this label to get a comment to choose whether to merge the PR to production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants