Skip to content

Fix for Screen Resolution Issue Winium crutius fails to perform click… #75

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
vinuthamikkilineni wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix for Screen Resolution Issue Winium crutius fails to perform click… #75

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
56 changes: 49 additions & 7 deletions src/Winium.Cruciatus/Helpers/AutomationElementHelper.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ namespace Winium.Cruciatus.Helpers
using System.Windows;
using System.Windows.Automation;
using System.Xml.XPath;
using NLog;

using Winium.Cruciatus.Helpers.XPath;

Expand All @@ -17,12 +18,31 @@ namespace Winium.Cruciatus.Helpers

internal static class AutomationElementHelper
{
public struct CursorPoint
{
public int X;
public int Y;
}
public enum DeviceCap
{

LOGPIXELSX = 88,
LOGPIXELSY = 90
}
#region Methods
static readonly Logger Logger = CruciatusFactory.Logger;

[System.Runtime.InteropServices.DllImport("user32.dll")]
internal static extern bool GetPhysicalCursorPos(ref CursorPoint lpPoint);
[System.Runtime.InteropServices.DllImport("gdi32.dll")]
static extern int GetDeviceCaps(IntPtr hdc, int nIndex);

static readonly int DEFAULTDPI = 96;

internal static IEnumerable<AutomationElement> FindAll(
AutomationElement parent,
TreeScope scope,
Condition condition,
AutomationElement parent,
TreeScope scope,
Condition condition,
int timeout)
{
var dtn = DateTime.Now.AddMilliseconds(timeout);
Expand Down Expand Up @@ -73,9 +93,9 @@ internal static AutomationElement FindFirst(AutomationElement parent, TreeScope
}

internal static AutomationElement FindFirst(
AutomationElement parent,
TreeScope scope,
Condition condition,
AutomationElement parent,
TreeScope scope,
Condition condition,
int timeout)
{
var dtn = DateTime.Now.AddMilliseconds(timeout);
Expand All @@ -93,6 +113,16 @@ internal static AutomationElement FindFirst(
return null;
}

static void getScreenScalingFactor(ref float ScreenScalingFactorX, ref float ScreenScalingFactorY)
{
System.Drawing.Graphics g = System.Drawing.Graphics.FromHwnd(IntPtr.Zero);
IntPtr desktop = g.GetHdc();
int Xdpi = GetDeviceCaps(desktop, (int)DeviceCap.LOGPIXELSX);
int Ydpi = GetDeviceCaps(desktop, (int)DeviceCap.LOGPIXELSY);
ScreenScalingFactorX = (float)Xdpi / (float)DEFAULTDPI;
ScreenScalingFactorY = (float)Ydpi / (float)DEFAULTDPI;
}

internal static bool TryGetBoundingRectangleCenter(AutomationElement element, out Point point)
{
var rect = element.Current.BoundingRectangle;
Expand All @@ -103,10 +133,21 @@ internal static bool TryGetBoundingRectangleCenter(AutomationElement element, ou
}

point = rect.Location;
point.Offset(rect.Width / 2, rect.Height / 2);
float ScreenScalingFactorX = 1;
float ScreenScalingFactorY = 1;
getScreenScalingFactor(ref ScreenScalingFactorX, ref ScreenScalingFactorY);
point.X = rect.X / ScreenScalingFactorX;
point.Y = rect.Y / ScreenScalingFactorY;

CruciatusFactory.Logger.Debug("BoundingRectangle location before scaling X co-ordinate:" + rect.X + "Y co-ordinate: " + rect.Y);
CruciatusFactory.Logger.Debug("BoundingRectangle Points after scaling X co-ordinate:" + point.X + "Y co-ordinate: " + point.Y);
CruciatusFactory.Logger.Debug("BoundingRectangle width :" + rect.Width);
CruciatusFactory.Logger.Debug("BoundingRectangle Height :" + rect.Height);
point.Offset((rect.Width / ScreenScalingFactorX) / 2, (rect.Height / ScreenScalingFactorY) / 2);
return true;
}


internal static bool TryGetClickablePoint(AutomationElement element, out Point point)
{
return element.TryGetClickablePoint(out point);
Expand All @@ -115,3 +156,4 @@ internal static bool TryGetClickablePoint(AutomationElement element, out Point p
#endregion
}
}