javier/Wino-Mail
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixing some IsBusy corner cases.

Browse Source
This commit is contained in:
Burak Kaan Köse
2026-03-01 12:40:12 +01:00
parent 211faff750
commit 2c9351f551
5 changed files with 61 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Wino.Mail.WinUI/Controls/MailItemDisplayInformationControl.xaml.cs
+50 -5
View File
@@ -1,4 +1,5 @@
using System;
using System.ComponentModel;
using System.Linq;
using System.Numerics;
using CommunityToolkit.WinUI;
@@ -26,6 +27,8 @@ public sealed partial class MailItemDisplayInformationControl : UserControl
private Compositor? _compositor;
private Visual? _contentVisual;
private ScalarKeyFrameAnimation? _opacityAnimation;
private SpriteVisual? _leftBackgroundVisual;
private INotifyPropertyChanged? _actionItemPropertySource;
[GeneratedDependencyProperty(DefaultValue = MailListDisplayMode.Spacious)]
public partial MailListDisplayMode DisplayMode { get; set; }
@@ -83,8 +86,8 @@ public sealed partial class MailItemDisplayInformationControl : UserControl
var compositor = this.Visual().Compositor;
var leftBackgroundVisual = compositor.CreateSpriteVisual();
RootContainerVisualWrapper.SetChildVisual(leftBackgroundVisual);
_leftBackgroundVisual = compositor.CreateSpriteVisual();
RootContainerVisualWrapper.SetChildVisual(_leftBackgroundVisual);
MainContentContainer.EnableImplicitAnimation(VisualPropertyType.Offset, 400);
RootContainer.EnableImplicitAnimation(VisualPropertyType.Offset, 400);
@@ -92,8 +95,6 @@ public sealed partial class MailItemDisplayInformationControl : UserControl
ContentStackpanel.EnableImplicitAnimation(VisualPropertyType.Offset, 400);
IconsContainer.EnableImplicitAnimation(VisualPropertyType.Offset, 400);
RootContainerVisualWrapper.SizeChanged += (s, e) => leftBackgroundVisual.Size = e.NewSize.ToVector2();
// Initialize shimmer effect compositor
_compositor = this.Visual().Compositor;
}
@@ -108,6 +109,23 @@ public sealed partial class MailItemDisplayInformationControl : UserControl
UpdateBusyAnimationState();
}
partial void OnActionItemPropertyChanged(DependencyPropertyChangedEventArgs e)
{
if (_actionItemPropertySource != null)
{
_actionItemPropertySource.PropertyChanged -= ActionItemPropertyChanged;
_actionItemPropertySource = null;
}
if (e.NewValue is INotifyPropertyChanged propertyChangedSource)
{
_actionItemPropertySource = propertyChangedSource;
_actionItemPropertySource.PropertyChanged += ActionItemPropertyChanged;
}
UpdateBusyAnimationState();
}
private void StartBusyAnimation()
{
if (_compositor == null) return;
@@ -144,7 +162,7 @@ public sealed partial class MailItemDisplayInformationControl : UserControl
private void UpdateBusyAnimationState()
{
if (MailItemInformation?.IsBusy == true)
if (ActionItem?.IsBusy == true)
{
StartBusyAnimation();
return;
@@ -153,6 +171,33 @@ public sealed partial class MailItemDisplayInformationControl : UserControl
StopBusyAnimation();
}
private void ActionItemPropertyChanged(object? sender, PropertyChangedEventArgs e)
{
if (string.IsNullOrEmpty(e.PropertyName) || e.PropertyName == nameof(IMailListItem.IsBusy))
{
UpdateBusyAnimationState();
}
}
private void OnUnloaded(object sender, RoutedEventArgs e)
{
if (_actionItemPropertySource != null)
{
_actionItemPropertySource.PropertyChanged -= ActionItemPropertyChanged;
_actionItemPropertySource = null;
}
StopBusyAnimation();
}
private void RootContainerVisualWrapperSizeChanged(object sender, SizeChangedEventArgs e)
{
if (_leftBackgroundVisual != null)
{
_leftBackgroundVisual.Size = e.NewSize.ToVector2();
}
}
private void ControlPointerEntered(object sender, Microsoft.UI.Xaml.Input.PointerRoutedEventArgs e)
{
if (IsHoverActionsEnabled)