From 2eb04c271909c2a54af23fa77ad99f96e7aeec74 Mon Sep 17 00:00:00 2001 From: robloo Date: Tue, 14 Mar 2023 12:08:20 -0400 Subject: [PATCH] Rename Popup.PlacementMode to Popup.Placement --- .../IntegrationTestApp/MainWindow.axaml.cs | 2 +- src/Avalonia.Controls/ContextMenu.cs | 4 +-- .../DateTimePickers/DatePicker.cs | 2 +- .../DateTimePickers/TimePicker.cs | 2 +- .../Flyouts/PopupFlyoutBase.cs | 14 ++++------ src/Avalonia.Controls/Primitives/Popup.cs | 28 +++++++++---------- .../SplitButton/SplitButton.cs | 2 +- .../Controls/DatePicker.xaml | 2 +- .../Controls/MenuItem.xaml | 2 +- .../Controls/TimePicker.xaml | 2 +- .../Controls/DatePicker.xaml | 2 +- .../Controls/MenuItem.xaml | 2 +- .../Controls/TimePicker.xaml | 2 +- .../Primitives/PopupRootTests.cs | 2 +- .../Primitives/PopupTests.cs | 20 ++++++------- 15 files changed, 43 insertions(+), 45 deletions(-) diff --git a/samples/IntegrationTestApp/MainWindow.axaml.cs b/samples/IntegrationTestApp/MainWindow.axaml.cs index 19eb1d64b0..b0b06303a4 100644 --- a/samples/IntegrationTestApp/MainWindow.axaml.cs +++ b/samples/IntegrationTestApp/MainWindow.axaml.cs @@ -158,7 +158,7 @@ namespace IntegrationTestApp var popup = new Popup { WindowManagerAddShadowHint = false, - PlacementMode = PlacementMode.AnchorAndGravity, + Placement = PlacementMode.AnchorAndGravity, PlacementAnchor = PopupAnchor.Top, PlacementGravity = PopupGravity.Bottom, Width= 200, diff --git a/src/Avalonia.Controls/ContextMenu.cs b/src/Avalonia.Controls/ContextMenu.cs index 0e7d31967c..d1af361867 100644 --- a/src/Avalonia.Controls/ContextMenu.cs +++ b/src/Avalonia.Controls/ContextMenu.cs @@ -58,7 +58,7 @@ namespace Avalonia.Controls /// Defines the property. /// public static readonly StyledProperty PlacementModeProperty = - Popup.PlacementModeProperty.AddOwner(); + Popup.PlacementProperty.AddOwner(); /// /// Defines the property. @@ -343,7 +343,7 @@ namespace Avalonia.Controls ((ISetLogicalParent)_popup).SetParent(control); } - _popup.PlacementMode = !requestedByPointer && PlacementMode == PlacementMode.Pointer + _popup.Placement = !requestedByPointer && PlacementMode == PlacementMode.Pointer ? PlacementMode.Bottom : PlacementMode; diff --git a/src/Avalonia.Controls/DateTimePickers/DatePicker.cs b/src/Avalonia.Controls/DateTimePickers/DatePicker.cs index 118183102a..f3faef26dd 100644 --- a/src/Avalonia.Controls/DateTimePickers/DatePicker.cs +++ b/src/Avalonia.Controls/DateTimePickers/DatePicker.cs @@ -389,7 +389,7 @@ namespace Avalonia.Controls _presenter.Date = SelectedDate ?? DateTimeOffset.Now; - _popup.PlacementMode = PlacementMode.AnchorAndGravity; + _popup.Placement = PlacementMode.AnchorAndGravity; _popup.PlacementAnchor = Primitives.PopupPositioning.PopupAnchor.Bottom; _popup.PlacementGravity = Primitives.PopupPositioning.PopupGravity.Bottom; _popup.PlacementConstraintAdjustment = Primitives.PopupPositioning.PopupPositionerConstraintAdjustment.SlideY; diff --git a/src/Avalonia.Controls/DateTimePickers/TimePicker.cs b/src/Avalonia.Controls/DateTimePickers/TimePicker.cs index 2f49a44b8c..0046e1de52 100644 --- a/src/Avalonia.Controls/DateTimePickers/TimePicker.cs +++ b/src/Avalonia.Controls/DateTimePickers/TimePicker.cs @@ -255,7 +255,7 @@ namespace Avalonia.Controls _presenter.Time = SelectedTime ?? DateTime.Now.TimeOfDay; - _popup.PlacementMode = PlacementMode.AnchorAndGravity; + _popup.Placement = PlacementMode.AnchorAndGravity; _popup.PlacementAnchor = Primitives.PopupPositioning.PopupAnchor.Bottom; _popup.PlacementGravity = Primitives.PopupPositioning.PopupGravity.Bottom; _popup.PlacementConstraintAdjustment = Primitives.PopupPositioning.PopupPositionerConstraintAdjustment.SlideY; diff --git a/src/Avalonia.Controls/Flyouts/PopupFlyoutBase.cs b/src/Avalonia.Controls/Flyouts/PopupFlyoutBase.cs index a3d05a34b7..7b78bacd5f 100644 --- a/src/Avalonia.Controls/Flyouts/PopupFlyoutBase.cs +++ b/src/Avalonia.Controls/Flyouts/PopupFlyoutBase.cs @@ -14,9 +14,9 @@ namespace Avalonia.Controls.Primitives { public abstract class PopupFlyoutBase : FlyoutBase, IPopupHostProvider { - /// + /// public static readonly StyledProperty PlacementProperty = - Popup.PlacementModeProperty.AddOwner(); + Popup.PlacementProperty.AddOwner(); /// public static readonly StyledProperty HorizontalOffsetProperty = @@ -64,15 +64,13 @@ namespace Avalonia.Controls.Primitives protected Popup Popup => _popupLazy.Value; - /// - /// Gets or sets the desired placement. - /// + /// public PlacementMode Placement { get => GetValue(PlacementProperty); set => SetValue(PlacementProperty, value); } - + /// public PopupGravity PlacementGravity { @@ -423,11 +421,11 @@ namespace Avalonia.Controls.Primitives Popup.PlacementGravity = PlacementGravity; if (showAtPointer) { - Popup.PlacementMode = PlacementMode.Pointer; + Popup.Placement = PlacementMode.Pointer; } else { - Popup.PlacementMode = Placement; + Popup.Placement = Placement; Popup.PlacementConstraintAdjustment = PopupPositioning.PopupPositionerConstraintAdjustment.SlideX | PopupPositioning.PopupPositionerConstraintAdjustment.SlideY; diff --git a/src/Avalonia.Controls/Primitives/Popup.cs b/src/Avalonia.Controls/Primitives/Popup.cs index 0c6c434713..acfd37244e 100644 --- a/src/Avalonia.Controls/Primitives/Popup.cs +++ b/src/Avalonia.Controls/Primitives/Popup.cs @@ -66,10 +66,10 @@ namespace Avalonia.Controls.Primitives AvaloniaProperty.Register(nameof(PlacementGravity)); /// - /// Defines the property. + /// Defines the property. /// - public static readonly StyledProperty PlacementModeProperty = - AvaloniaProperty.Register(nameof(PlacementMode), defaultValue: PlacementMode.Bottom); + public static readonly StyledProperty PlacementProperty = + AvaloniaProperty.Register(nameof(Placement), defaultValue: PlacementMode.Bottom); /// /// Defines the property. @@ -205,7 +205,7 @@ namespace Avalonia.Controls.Primitives } /// - /// Gets or sets the anchor point on the when + /// Gets or sets the anchor point on the when /// is . /// public PopupAnchor PlacementAnchor @@ -226,7 +226,7 @@ namespace Avalonia.Controls.Primitives /// /// Gets or sets a value which defines in what direction the popup should open - /// when is . + /// when is . /// public PopupGravity PlacementGravity { @@ -235,17 +235,17 @@ namespace Avalonia.Controls.Primitives } /// - /// Gets or sets the placement mode of the popup in relation to the . + /// Gets or sets the desired placement of the popup in relation to the . /// - public PlacementMode PlacementMode + public PlacementMode Placement { - get { return GetValue(PlacementModeProperty); } - set { SetValue(PlacementModeProperty, value); } + get { return GetValue(PlacementProperty); } + set { SetValue(PlacementProperty, value); } } /// /// Gets or sets the the anchor rectangle within the parent that the popup will be placed - /// relative to when is . + /// relative to when is . /// /// /// The placement rect defines a rectangle relative to around @@ -404,7 +404,7 @@ namespace Avalonia.Controls.Primitives (x, handler) => x.LostFocus -= handler).DisposeWith(handlerCleanup); // Recalculate popup position on parent moved/resized, but not if placement was on pointer - if (PlacementMode != PlacementMode.Pointer) + if (Placement != PlacementMode.Pointer) { SubscribeToEventHandler>(window.PlatformImpl, WindowPositionChanged, (x, handler) => x.PositionChanged += handler, @@ -534,7 +534,7 @@ namespace Avalonia.Controls.Primitives UpdateHostSizing(_openState.PopupHost, _openState.TopLevel, _openState.PlacementTarget); } else if (change.Property == PlacementTargetProperty || - change.Property == PlacementModeProperty || + change.Property == PlacementProperty || change.Property == HorizontalOffsetProperty || change.Property == VerticalOffsetProperty || change.Property == PlacementAnchorProperty || @@ -567,7 +567,7 @@ namespace Avalonia.Controls.Primitives { popupHost.ConfigurePosition( placementTarget, - PlacementMode, + Placement, new Point(HorizontalOffset, VerticalOffset), PlacementAnchor, PlacementGravity, @@ -615,7 +615,7 @@ namespace Avalonia.Controls.Primitives return; _openState.PopupHost.ConfigurePosition( placementTarget, - PlacementMode, + Placement, new Point(HorizontalOffset, VerticalOffset), PlacementAnchor, PlacementGravity, diff --git a/src/Avalonia.Controls/SplitButton/SplitButton.cs b/src/Avalonia.Controls/SplitButton/SplitButton.cs index e790578675..7f10c632c5 100644 --- a/src/Avalonia.Controls/SplitButton/SplitButton.cs +++ b/src/Avalonia.Controls/SplitButton/SplitButton.cs @@ -172,7 +172,7 @@ namespace Avalonia.Controls flyout.Opened += Flyout_Opened; flyout.Closed += Flyout_Closed; - _flyoutPropertyChangedDisposable = flyout.GetPropertyChangedObservable(Popup.PlacementModeProperty).Subscribe(Flyout_PlacementPropertyChanged); + _flyoutPropertyChangedDisposable = flyout.GetPropertyChangedObservable(Popup.PlacementProperty).Subscribe(Flyout_PlacementPropertyChanged); } } diff --git a/src/Avalonia.Themes.Fluent/Controls/DatePicker.xaml b/src/Avalonia.Themes.Fluent/Controls/DatePicker.xaml index 28ec553660..e71e733eff 100644 --- a/src/Avalonia.Themes.Fluent/Controls/DatePicker.xaml +++ b/src/Avalonia.Themes.Fluent/Controls/DatePicker.xaml @@ -127,7 +127,7 @@ + Placement="Bottom"> diff --git a/src/Avalonia.Themes.Fluent/Controls/MenuItem.xaml b/src/Avalonia.Themes.Fluent/Controls/MenuItem.xaml index 72a7797bc3..5a10c62860 100644 --- a/src/Avalonia.Themes.Fluent/Controls/MenuItem.xaml +++ b/src/Avalonia.Themes.Fluent/Controls/MenuItem.xaml @@ -118,7 +118,7 @@ diff --git a/src/Avalonia.Themes.Fluent/Controls/TimePicker.xaml b/src/Avalonia.Themes.Fluent/Controls/TimePicker.xaml index 71ff0eaebc..39f7b0b224 100644 --- a/src/Avalonia.Themes.Fluent/Controls/TimePicker.xaml +++ b/src/Avalonia.Themes.Fluent/Controls/TimePicker.xaml @@ -151,7 +151,7 @@ WindowManagerAddShadowHint="False" IsLightDismissEnabled="True" PlacementTarget="{TemplateBinding}" - PlacementMode="Bottom"> + Placement="Bottom"> diff --git a/src/Avalonia.Themes.Simple/Controls/DatePicker.xaml b/src/Avalonia.Themes.Simple/Controls/DatePicker.xaml index 5c57240aea..6cf352518a 100644 --- a/src/Avalonia.Themes.Simple/Controls/DatePicker.xaml +++ b/src/Avalonia.Themes.Simple/Controls/DatePicker.xaml @@ -142,7 +142,7 @@ diff --git a/src/Avalonia.Themes.Simple/Controls/MenuItem.xaml b/src/Avalonia.Themes.Simple/Controls/MenuItem.xaml index 6056938938..7ec666eaf9 100644 --- a/src/Avalonia.Themes.Simple/Controls/MenuItem.xaml +++ b/src/Avalonia.Themes.Simple/Controls/MenuItem.xaml @@ -65,7 +65,7 @@ IsLightDismissEnabled="False" IsOpen="{TemplateBinding IsSubMenuOpen, Mode=TwoWay}" - PlacementMode="Right"> + Placement="Right"> diff --git a/src/Avalonia.Themes.Simple/Controls/TimePicker.xaml b/src/Avalonia.Themes.Simple/Controls/TimePicker.xaml index 58066bb9a5..5acbc41a68 100644 --- a/src/Avalonia.Themes.Simple/Controls/TimePicker.xaml +++ b/src/Avalonia.Themes.Simple/Controls/TimePicker.xaml @@ -156,7 +156,7 @@ diff --git a/tests/Avalonia.Controls.UnitTests/Primitives/PopupRootTests.cs b/tests/Avalonia.Controls.UnitTests/Primitives/PopupRootTests.cs index 788e6a48bb..fb88c98343 100644 --- a/tests/Avalonia.Controls.UnitTests/Primitives/PopupRootTests.cs +++ b/tests/Avalonia.Controls.UnitTests/Primitives/PopupRootTests.cs @@ -67,7 +67,7 @@ namespace Avalonia.Controls.UnitTests.Primitives using (UnitTestApplication.Start(TestServices.StyledWindow)) { var window = new Window(); - var target = new Popup {PlacementMode = PlacementMode.Pointer}; + var target = new Popup {Placement = PlacementMode.Pointer}; var child = new Control(); window.Content = target; diff --git a/tests/Avalonia.Controls.UnitTests/Primitives/PopupTests.cs b/tests/Avalonia.Controls.UnitTests/Primitives/PopupTests.cs index bc1225e0e8..c3d76954e4 100644 --- a/tests/Avalonia.Controls.UnitTests/Primitives/PopupTests.cs +++ b/tests/Avalonia.Controls.UnitTests/Primitives/PopupTests.cs @@ -207,7 +207,7 @@ namespace Avalonia.Controls.UnitTests.Primitives { using (CreateServices()) { - var target = new Popup() {PlacementMode = PlacementMode.Pointer}; + var target = new Popup() {Placement = PlacementMode.Pointer}; var root = PreparedWindow(target); target.Open(); @@ -226,7 +226,7 @@ namespace Avalonia.Controls.UnitTests.Primitives using (CreateServices()) { var window = PreparedWindow(); - var target = new Popup() {PlacementMode = PlacementMode.Pointer}; + var target = new Popup() {Placement = PlacementMode.Pointer}; window.Content = target; @@ -249,7 +249,7 @@ namespace Avalonia.Controls.UnitTests.Primitives using (CreateServices()) { var window = PreparedWindow(); - var target = new Popup() {PlacementMode = PlacementMode.Pointer}; + var target = new Popup() {Placement = PlacementMode.Pointer}; window.Content = target; window.ApplyTemplate(); @@ -274,7 +274,7 @@ namespace Avalonia.Controls.UnitTests.Primitives using (CreateServices()) { var window = PreparedWindow(); - var target = new Popup() { PlacementMode = PlacementMode.Pointer }; + var target = new Popup() { Placement = PlacementMode.Pointer }; window.Content = target; window.ApplyTemplate(); @@ -742,7 +742,7 @@ namespace Avalonia.Controls.UnitTests.Primitives { Width = 400, Height = 200, - PlacementMode = PlacementMode.Pointer + Placement = PlacementMode.Pointer }; var window = PreparedWindow(popup); window.Show(); @@ -791,7 +791,7 @@ namespace Avalonia.Controls.UnitTests.Primitives var popup = new Popup() { PlacementTarget = placementTarget, - PlacementMode = PlacementMode.Bottom, + Placement = PlacementMode.Bottom, Width = 10, Height = 10 }; @@ -852,7 +852,7 @@ namespace Avalonia.Controls.UnitTests.Primitives var popup = new Popup() { PlacementTarget = placementTarget, - PlacementMode = PlacementMode.Pointer, + Placement = PlacementMode.Pointer, Width = 10, Height = 10 }; @@ -907,7 +907,7 @@ namespace Avalonia.Controls.UnitTests.Primitives var popup = new Popup() { PlacementTarget = placementTarget, - PlacementMode = PlacementMode.Bottom, + Placement = PlacementMode.Bottom, Width = 10, Height = 10 }; @@ -967,7 +967,7 @@ namespace Avalonia.Controls.UnitTests.Primitives var popup = new Popup() { PlacementTarget = placementTarget, - PlacementMode = PlacementMode.Pointer, + Placement = PlacementMode.Pointer, Width = 10, Height = 10 }; @@ -1020,7 +1020,7 @@ namespace Avalonia.Controls.UnitTests.Primitives Width = 20, Height = 20, PlacementTarget = parentPopup, - PlacementMode = PlacementMode.AnchorAndGravity, + Placement = PlacementMode.AnchorAndGravity, PlacementAnchor = PopupAnchor.TopLeft, PlacementGravity = PopupGravity.BottomRight };