From 4c4199f9d560bb46de9770f902e5505af5e7e1be Mon Sep 17 00:00:00 2001 From: Michael Babienco Date: Wed, 29 Jul 2020 12:28:21 -0400 Subject: [PATCH] Use default param instead of setting _resetQueued to true on menu set --- src/Avalonia.Native/AvaloniaNativeMenuExporter.cs | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/Avalonia.Native/AvaloniaNativeMenuExporter.cs b/src/Avalonia.Native/AvaloniaNativeMenuExporter.cs index 07fe2b7d72..5a99249fee 100644 --- a/src/Avalonia.Native/AvaloniaNativeMenuExporter.cs +++ b/src/Avalonia.Native/AvaloniaNativeMenuExporter.cs @@ -39,8 +39,7 @@ namespace Avalonia.Native public void SetNativeMenu(NativeMenu menu) { _menu = menu == null ? new NativeMenu() : menu; - _resetQueued = true; // we've reset the menu to something new, so make sure that a reset is queued - DoLayoutReset(); + DoLayoutReset(true); } internal void UpdateIfNeeded() @@ -74,9 +73,9 @@ namespace Avalonia.Native return result; } - private void DoLayoutReset() + private void DoLayoutReset(bool forceUpdate = false) { - if (_resetQueued) + if (_resetQueued || forceUpdate) { _resetQueued = false; @@ -109,7 +108,7 @@ namespace Avalonia.Native if (_resetQueued) return; _resetQueued = true; - Dispatcher.UIThread.Post(DoLayoutReset, DispatcherPriority.Background); + Dispatcher.UIThread.Post(() => DoLayoutReset(), DispatcherPriority.Background); } private void SetMenu(NativeMenu menu)