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

Merge pull request #167 from Tiktack/wino165-commandbar-issues

Browse Source 
Fix commend bar issues
This commit is contained in:
Burak Kaan Köse
2024-04-28 17:50:02 +02:00
committed by GitHub
parent 75863faf58 2c4c7586b7
commit e98f6997ca
4 changed files with 24 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Wino.Mail.ViewModels/MailRenderingPageViewModel.cs
View File

@@ -462,7 +462,7 @@ namespace Wino.Mail.ViewModels
MenuItems.Clear();
// Add light/dark editor theme switch.
if (_underlyingThemeService.IsUnderlyingThemeDark())
if (IsDarkWebviewRenderer)
MenuItems.Add(MailOperationMenuItem.Create(MailOperation.LightEditor));
else
MenuItems.Add(MailOperationMenuItem.Create(MailOperation.DarkEditor));

47
Wino.Mail/Behaviors/BindableCommandBarBehavior.cs
View File

@@ -1,7 +1,7 @@
using Microsoft.Xaml.Interactivity;
using System.Collections;
using System.Collections;
using System.Collections.Specialized;
using System.Windows.Input;
using Microsoft.Xaml.Interactivity;
using Windows.UI.Xaml;
using Windows.UI.Xaml.Controls;
using Windows.UI.Xaml.Controls.Primitives;
@@ -98,42 +98,37 @@ namespace Wino.Behaviors
{
if (command is MailOperationMenuItem mailOperationMenuItem)
{
ICommandBarElement menuItem = null;
if (mailOperationMenuItem.Operation == Core.Domain.Enums.MailOperation.Seperator)
{
var seperator = new AppBarSeparator();
if (mailOperationMenuItem.IsSecondaryMenuPreferred)
{
AssociatedObject.SecondaryCommands.Add(seperator);
}
else
{
AssociatedObject.PrimaryCommands.Add(seperator);
}
menuItem = new AppBarSeparator();
}
else
{
var menuItem = new AppBarButton()
var label = XamlHelpers.GetOperationString(mailOperationMenuItem.Operation);
menuItem = new AppBarButton
{
Icon = new WinoFontIcon() { Glyph = ControlConstants.WinoIconFontDictionary[XamlHelpers.GetWinoIconGlyph(mailOperationMenuItem.Operation)] },
Label = XamlHelpers.GetOperationString(mailOperationMenuItem.Operation),
Label = label,
LabelPosition = string.IsNullOrWhiteSpace(label) ? CommandBarLabelPosition.Collapsed : CommandBarLabelPosition.Default,
DataContext = mailOperationMenuItem,
};
menuItem.Click -= Button_Click;
menuItem.Click += Button_Click;
((AppBarButton)menuItem).Click -= Button_Click;
((AppBarButton)menuItem).Click += Button_Click;
}
if (mailOperationMenuItem.IsSecondaryMenuPreferred)
{
AssociatedObject.SecondaryCommands.Add(menuItem);
}
else
{
AssociatedObject.PrimaryCommands.Add(menuItem);
}
if (mailOperationMenuItem.IsSecondaryMenuPreferred)
{
AssociatedObject.SecondaryCommands.Add(menuItem);
}
else
{
AssociatedObject.PrimaryCommands.Add(menuItem);
}
}
//if (dependencyObject is ICommandBarElement icommandBarElement)
//{
// if (dependencyObject is ButtonBase button)
@@ -144,7 +139,7 @@ namespace Wino.Behaviors
// if (command is MailOperationMenuItem mailOperationMenuItem)
// {
// }
//}
}

1
Wino.Mail/Views/MailRenderingPage.xaml
View File

@@ -150,7 +150,6 @@
x:Name="RendererBar"
Grid.Row="0"
Margin="8,6,0,0"
IsSticky="True"
DefaultLabelPosition="Right"
HorizontalContentAlignment="Stretch"
OverflowButtonVisibility="Auto"

3
Wino.Mail/Views/MailRenderingPage.xaml.cs
View File

@@ -1,4 +1,5 @@
using System;
using System.Linq;
using System.Threading.Tasks;
using CommunityToolkit.Mvvm.Messaging;
using Microsoft.AppCenter.Crashes;
@@ -218,7 +219,7 @@ namespace Wino.Views
private void BarDynamicOverflowChanging(CommandBar sender, DynamicOverflowItemsChangingEventArgs args)
{
if (args.Action == CommandBarDynamicOverflowAction.AddingToOverflow)
if (args.Action == CommandBarDynamicOverflowAction.AddingToOverflow || sender.SecondaryCommands.Any())
sender.OverflowButtonVisibility = CommandBarOverflowButtonVisibility.Visible;
else
sender.OverflowButtonVisibility = CommandBarOverflowButtonVisibility.Collapsed;