Merge pull request #923 from jackBonadies/development

Update readonly numberbox to prohibit changing values via keyboard / buttons
lepoco · Feb 3, 2024 · 7b6cc72 · 7b6cc72
2 parents 2bfbb99 + 5709edd
commit 7b6cc72
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/Wpf.Ui/Controls/NumberBox/NumberBox.cs b/src/Wpf.Ui/Controls/NumberBox/NumberBox.cs
@@ -264,6 +264,11 @@ protected override void OnKeyUp(KeyEventArgs e)
     {
         base.OnKeyUp(e);
 
+        if(IsReadOnly)
+        {
+            return;
+        }
+
         switch (e.Key)
         {
             case Key.PageUp:

diff --git a/src/Wpf.Ui/Controls/NumberBox/NumberBox.xaml b/src/Wpf.Ui/Controls/NumberBox/NumberBox.xaml
@@ -235,6 +235,7 @@
                             <Setter TargetName="ContentBorder" Property="Background" Value="{DynamicResource TextControlBackgroundPointerOver}" />
                         </MultiTrigger>
                         <Trigger Property="IsReadOnly" Value="True">
+                            <Setter Property="SpinButtonPlacementMode" Value="Hidden" />
                             <Setter TargetName="ClearButton" Property="Visibility" Value="Collapsed" />
                             <Setter TargetName="ClearButton" Property="Margin" Value="0" />
                         </Trigger>