mirror of
https://github.com/microsoft/PowerToys.git
synced 2025-12-16 11:48:06 +01:00
[ColorPicker]Fix "Failed to get mouse position-1,1" warning (#20291)
* ColorPicker-17636: Fix warning "Failed to get ... mouse position-1,1" * retrigger checks * retrigger checks
This commit is contained in:
@@ -19,7 +19,6 @@ namespace ColorPicker.Behaviors
|
|||||||
|
|
||||||
private void AssociatedObject_Loaded(object sender, RoutedEventArgs e)
|
private void AssociatedObject_Loaded(object sender, RoutedEventArgs e)
|
||||||
{
|
{
|
||||||
Appear();
|
|
||||||
AssociatedObject.IsVisibleChanged += AssociatedObject_IsVisibleChanged;
|
AssociatedObject.IsVisibleChanged += AssociatedObject_IsVisibleChanged;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -19,5 +19,7 @@ namespace ColorPicker.Mouse
|
|||||||
event MouseUpEventHandler OnMouseDown;
|
event MouseUpEventHandler OnMouseDown;
|
||||||
|
|
||||||
System.Windows.Point CurrentPosition { get; }
|
System.Windows.Point CurrentPosition { get; }
|
||||||
|
|
||||||
|
Color CurrentColor { get; }
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -42,6 +42,8 @@ namespace ColorPicker.Mouse
|
|||||||
_mouseHook = new MouseHook();
|
_mouseHook = new MouseHook();
|
||||||
_userSettings = userSettings;
|
_userSettings = userSettings;
|
||||||
_userSettings.CopiedColorRepresentation.PropertyChanged += CopiedColorRepresentation_PropertyChanged;
|
_userSettings.CopiedColorRepresentation.PropertyChanged += CopiedColorRepresentation_PropertyChanged;
|
||||||
|
_previousMousePosition = GetCursorPosition();
|
||||||
|
_previousColor = GetPixelColor(_previousMousePosition);
|
||||||
}
|
}
|
||||||
|
|
||||||
public event EventHandler<Color> MouseColorChanged;
|
public event EventHandler<Color> MouseColorChanged;
|
||||||
@@ -60,6 +62,14 @@ namespace ColorPicker.Mouse
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public Color CurrentColor
|
||||||
|
{
|
||||||
|
get
|
||||||
|
{
|
||||||
|
return _previousColor;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
private void Timer_Tick(object sender, EventArgs e)
|
private void Timer_Tick(object sender, EventArgs e)
|
||||||
{
|
{
|
||||||
UpdateMouseInfo();
|
UpdateMouseInfo();
|
||||||
|
|||||||
@@ -67,6 +67,7 @@ namespace ColorPicker.ViewModels
|
|||||||
|
|
||||||
if (mouseInfoProvider != null)
|
if (mouseInfoProvider != null)
|
||||||
{
|
{
|
||||||
|
SetColorDetails(mouseInfoProvider.CurrentColor);
|
||||||
mouseInfoProvider.MouseColorChanged += Mouse_ColorChanged;
|
mouseInfoProvider.MouseColorChanged += Mouse_ColorChanged;
|
||||||
mouseInfoProvider.OnMouseDown += MouseInfoProvider_OnMouseDown;
|
mouseInfoProvider.OnMouseDown += MouseInfoProvider_OnMouseDown;
|
||||||
mouseInfoProvider.OnMouseWheel += MouseInfoProvider_OnMouseWheel;
|
mouseInfoProvider.OnMouseWheel += MouseInfoProvider_OnMouseWheel;
|
||||||
@@ -132,9 +133,7 @@ namespace ColorPicker.ViewModels
|
|||||||
/// <param name="color">The new <see cref="Color"/> under the mouse cursor</param>
|
/// <param name="color">The new <see cref="Color"/> under the mouse cursor</param>
|
||||||
private void Mouse_ColorChanged(object sender, System.Drawing.Color color)
|
private void Mouse_ColorChanged(object sender, System.Drawing.Color color)
|
||||||
{
|
{
|
||||||
ColorBrush = new SolidColorBrush(Color.FromArgb(color.A, color.R, color.G, color.B));
|
SetColorDetails(color);
|
||||||
ColorText = ColorRepresentationHelper.GetStringRepresentation(color, _userSettings.CopiedColorRepresentation.Value);
|
|
||||||
ColorName = ColorNameHelper.GetColorName(color);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
@@ -172,6 +171,13 @@ namespace ColorPicker.ViewModels
|
|||||||
return color.A + "|" + color.R + "|" + color.G + "|" + color.B;
|
return color.A + "|" + color.R + "|" + color.G + "|" + color.B;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private void SetColorDetails(System.Drawing.Color color)
|
||||||
|
{
|
||||||
|
ColorBrush = new SolidColorBrush(Color.FromArgb(color.A, color.R, color.G, color.B));
|
||||||
|
ColorText = ColorRepresentationHelper.GetStringRepresentation(color, _userSettings.CopiedColorRepresentation.Value);
|
||||||
|
ColorName = ColorNameHelper.GetColorName(color);
|
||||||
|
}
|
||||||
|
|
||||||
/// <summary>
|
/// <summary>
|
||||||
/// Tell the color picker that the user have used the mouse wheel
|
/// Tell the color picker that the user have used the mouse wheel
|
||||||
/// </summary>
|
/// </summary>
|
||||||
|
|||||||
Reference in New Issue
Block a user