From fad4f5b5c9f819a4b917c55e9c6ad397cc418401 Mon Sep 17 00:00:00 2001
From: Jeremy Wu <jeremy24wu@gmail.com>
Date: Fri, 17 Jul 2020 21:14:15 +1000
Subject: [PATCH] fix clicking twice to expand issue

event was wired to the same expand method cause calling it twice
---
 .../Views/ExplorerSettings.xaml                      |  8 ++++----
 .../Views/ExplorerSettings.xaml.cs                   | 12 ++++++++++++
 2 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/Plugins/Flow.Launcher.Plugin.Explorer/Views/ExplorerSettings.xaml b/Plugins/Flow.Launcher.Plugin.Explorer/Views/ExplorerSettings.xaml
index dc9d83771..9d6f4976e 100644
--- a/Plugins/Flow.Launcher.Plugin.Explorer/Views/ExplorerSettings.xaml
+++ b/Plugins/Flow.Launcher.Plugin.Explorer/Views/ExplorerSettings.xaml
@@ -1,4 +1,4 @@
-<UserControl x:Class="Flow.Launcher.Plugin.Explorer.Views.ExplorerSettings"
+<UserControl x:Class="Flow.Launcher.Plugin.Explorer.Views.ExplorerSettings"
              xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" 
@@ -36,12 +36,12 @@
         <ScrollViewer Margin="20 35 0 0" HorizontalScrollBarVisibility="Hidden" Grid.Row="0" VerticalScrollBarVisibility="Auto">
             <StackPanel>
                 <Expander Name="expActionKeywords" Header="{DynamicResource plugin_explorer_manageactionkeywords_header}"
-                          Expanded="expActionKeywords_Click" Collapsed="expActionKeywords_Click">
+                          Expanded="expActionKeywords_Click" Collapsed="expActionKeywords_Collapsed">
                     <ListView x:Name="lbxActionKeywords"
                         ItemTemplate="{StaticResource ListViewActionKeywords}"/>
                 </Expander>
                 <Expander Name="expFolderLinks" Header="{DynamicResource plugin_explorer_quickfolderaccess_header}"
-                          Expanded="expFolderLinks_Click" Collapsed="expFolderLinks_Click"
+                          Expanded="expFolderLinks_Click" Collapsed="expFolderLinks_Collapsed"
                           Margin="0 10 0 0">
                     <ListView
                         x:Name="lbxFolderLinks" AllowDrop="True"
@@ -50,7 +50,7 @@
                         ItemTemplate="{StaticResource ListViewTemplateFolderLinks}"/>
                 </Expander>
                 <Expander x:Name="expExcludedPaths" Header="{DynamicResource plugin_explorer_indexsearchexcludedpaths_header}" 
-                          Expanded="expExcludedPaths_Click" Collapsed="expExcludedPaths_Click"
+                          Expanded="expExcludedPaths_Click"
                           Margin="0 10 0 0">
                     <ListView
                         x:Name="lbxExcludedPaths" AllowDrop="True"
diff --git a/Plugins/Flow.Launcher.Plugin.Explorer/Views/ExplorerSettings.xaml.cs b/Plugins/Flow.Launcher.Plugin.Explorer/Views/ExplorerSettings.xaml.cs
index 23a28497f..51c0cc905 100644
--- a/Plugins/Flow.Launcher.Plugin.Explorer/Views/ExplorerSettings.xaml.cs
+++ b/Plugins/Flow.Launcher.Plugin.Explorer/Views/ExplorerSettings.xaml.cs
@@ -93,6 +93,12 @@ namespace Flow.Launcher.Plugin.Explorer.Views
 
         }
 
+        private void expActionKeywords_Collapsed(object sender, RoutedEventArgs e)
+        {
+            if (!expActionKeywords.IsExpanded)
+                expActionKeywords.Height = Double.NaN;
+        }
+
         private void expFolderLinks_Click(object sender, RoutedEventArgs e)
         {
             if (expFolderLinks.IsExpanded)
@@ -107,6 +113,12 @@ namespace Flow.Launcher.Plugin.Explorer.Views
             RefreshView();
         }
 
+        private void expFolderLinks_Collapsed(object sender, RoutedEventArgs e)
+        {
+            if (!expFolderLinks.IsExpanded)
+                expFolderLinks.Height = Double.NaN;
+        }
+
         private void expExcludedPaths_Click(object sender, RoutedEventArgs e)
         {
             if (expExcludedPaths.IsExpanded)