Fix commend bar issues #165
This commit is contained in:
@@ -462,7 +462,7 @@ namespace Wino.Mail.ViewModels
|
|||||||
MenuItems.Clear();
|
MenuItems.Clear();
|
||||||
|
|
||||||
// Add light/dark editor theme switch.
|
// Add light/dark editor theme switch.
|
||||||
if (_underlyingThemeService.IsUnderlyingThemeDark())
|
if (IsDarkWebviewRenderer)
|
||||||
MenuItems.Add(MailOperationMenuItem.Create(MailOperation.LightEditor));
|
MenuItems.Add(MailOperationMenuItem.Create(MailOperation.LightEditor));
|
||||||
else
|
else
|
||||||
MenuItems.Add(MailOperationMenuItem.Create(MailOperation.DarkEditor));
|
MenuItems.Add(MailOperationMenuItem.Create(MailOperation.DarkEditor));
|
||||||
|
|||||||
@@ -1,7 +1,7 @@
|
|||||||
using Microsoft.Xaml.Interactivity;
|
using System.Collections;
|
||||||
using System.Collections;
|
|
||||||
using System.Collections.Specialized;
|
using System.Collections.Specialized;
|
||||||
using System.Windows.Input;
|
using System.Windows.Input;
|
||||||
|
using Microsoft.Xaml.Interactivity;
|
||||||
using Windows.UI.Xaml;
|
using Windows.UI.Xaml;
|
||||||
using Windows.UI.Xaml.Controls;
|
using Windows.UI.Xaml.Controls;
|
||||||
using Windows.UI.Xaml.Controls.Primitives;
|
using Windows.UI.Xaml.Controls.Primitives;
|
||||||
@@ -98,39 +98,34 @@ namespace Wino.Behaviors
|
|||||||
{
|
{
|
||||||
if (command is MailOperationMenuItem mailOperationMenuItem)
|
if (command is MailOperationMenuItem mailOperationMenuItem)
|
||||||
{
|
{
|
||||||
|
ICommandBarElement menuItem = null;
|
||||||
|
|
||||||
if (mailOperationMenuItem.Operation == Core.Domain.Enums.MailOperation.Seperator)
|
if (mailOperationMenuItem.Operation == Core.Domain.Enums.MailOperation.Seperator)
|
||||||
{
|
{
|
||||||
var seperator = new AppBarSeparator();
|
menuItem = new AppBarSeparator();
|
||||||
|
|
||||||
if (mailOperationMenuItem.IsSecondaryMenuPreferred)
|
|
||||||
{
|
|
||||||
AssociatedObject.SecondaryCommands.Add(seperator);
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
AssociatedObject.PrimaryCommands.Add(seperator);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
var menuItem = new AppBarButton()
|
var label = XamlHelpers.GetOperationString(mailOperationMenuItem.Operation);
|
||||||
|
menuItem = new AppBarButton
|
||||||
{
|
{
|
||||||
Icon = new WinoFontIcon() { Glyph = ControlConstants.WinoIconFontDictionary[XamlHelpers.GetWinoIconGlyph(mailOperationMenuItem.Operation)] },
|
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,
|
DataContext = mailOperationMenuItem,
|
||||||
};
|
};
|
||||||
|
|
||||||
menuItem.Click -= Button_Click;
|
((AppBarButton)menuItem).Click -= Button_Click;
|
||||||
menuItem.Click += Button_Click;
|
((AppBarButton)menuItem).Click += Button_Click;
|
||||||
|
}
|
||||||
|
|
||||||
if (mailOperationMenuItem.IsSecondaryMenuPreferred)
|
if (mailOperationMenuItem.IsSecondaryMenuPreferred)
|
||||||
{
|
{
|
||||||
AssociatedObject.SecondaryCommands.Add(menuItem);
|
AssociatedObject.SecondaryCommands.Add(menuItem);
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
AssociatedObject.PrimaryCommands.Add(menuItem);
|
AssociatedObject.PrimaryCommands.Add(menuItem);
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -150,12 +150,10 @@
|
|||||||
x:Name="RendererBar"
|
x:Name="RendererBar"
|
||||||
Grid.Row="0"
|
Grid.Row="0"
|
||||||
Margin="8,6,0,0"
|
Margin="8,6,0,0"
|
||||||
IsSticky="True"
|
IsSticky="False"
|
||||||
DefaultLabelPosition="Right"
|
DefaultLabelPosition="Right"
|
||||||
HorizontalContentAlignment="Stretch"
|
HorizontalContentAlignment="Stretch"
|
||||||
OverflowButtonVisibility="Auto"
|
OverflowButtonVisibility="Auto">
|
||||||
DynamicOverflowItemsChanging="BarDynamicOverflowChanging"
|
|
||||||
IsDynamicOverflowEnabled="True">
|
|
||||||
<interactivity:Interaction.Behaviors>
|
<interactivity:Interaction.Behaviors>
|
||||||
<local:BindableCommandBarBehavior
|
<local:BindableCommandBarBehavior
|
||||||
ItemClickedCommand="{x:Bind ViewModel.OperationClickedCommand}"
|
ItemClickedCommand="{x:Bind ViewModel.OperationClickedCommand}"
|
||||||
|
|||||||
Reference in New Issue
Block a user