Fix Hiding Master Page on MasterDetailPage
authorjh5.cho <jh5.cho@samsung.com>
Thu, 9 Feb 2017 07:11:33 +0000 (16:11 +0900)
committerKangho Hur <kangho.hur@samsung.com>
Mon, 10 Jul 2017 02:11:18 +0000 (11:11 +0900)
    - Fix to hide Master page which is caused by the focus issue
    - Remove private variable '_isPresented' to remove unnecessary steps

TASK=TCAAPI-2192

Change-Id: I0a5edc4622643a11c6afa33f58b86fc773a15915

Xamarin.Forms.Platform.Tizen/Native/MasterDetailPage.cs [changed mode: 0644->0755]

old mode 100644 (file)
new mode 100755 (executable)
index 5751aaf..1a10b70
@@ -69,11 +69,6 @@ namespace Xamarin.Forms.Platform.Tizen.Native
                EvasObject _mainWidget;
 
                /// <summary>
-               /// The <see cref="IsPresented"/> property value.
-               /// </summary>
-               bool _isPresented;
-
-               /// <summary>
                /// The <see cref="IsGestureEnabled"/> property value.
                /// </summary>
                bool _isGestureEnabled;
@@ -123,7 +118,7 @@ namespace Xamarin.Forms.Platform.Tizen.Native
                                Proportion = s_splitRatio,
                        };
 
-                       _drawer = new Panel(this)
+                       _drawer = new Panel(Forms.Context.MainWindow)
                        {
                                Direction = PanelDirection.Left,
                        };
@@ -131,12 +126,7 @@ namespace Xamarin.Forms.Platform.Tizen.Native
                        _drawer.SetScrollableArea(1.0);
                        _drawer.Toggled += (object sender, EventArgs e) =>
                        {
-                               IsPresented = _drawer.IsOpen;
-                       };
-
-                       IsPresentedChanged += (sender, e) =>
-                       {
-                               _drawer.IsOpen = IsPresented;
+                               IsPresentedChanged?.Invoke(this, EventArgs.Empty);
                        };
 
                        ConfigureLayout();
@@ -230,15 +220,14 @@ namespace Xamarin.Forms.Platform.Tizen.Native
                {
                        get
                        {
-                               return _isPresented;
+                               return _drawer.IsOpen;
                        }
 
                        set
                        {
-                               if (_isPresented != value)
+                               if (_drawer.IsOpen != value)
                                {
-                                       _isPresented = value;
-                                       IsPresentedChanged?.Invoke(this, EventArgs.Empty);
+                                       _drawer.IsOpen = value;
                                }
                        }
                }
@@ -348,12 +337,9 @@ namespace Xamarin.Forms.Platform.Tizen.Native
                        {
                                _drawer.SetContent(_masterCanvas, true);
                                PackEnd(_detailCanvas);
-                               PackEnd(_drawer);
-
                                _drawer.Show();
 
                                _mainWidget = _detailCanvas;
-
                                _drawer.IsOpen = IsPresented;
                        }