Hi Prasanta,
Now the changes look fine. However, I noticed that there is no testcase associated with this fix. Is that intentional? Thanks, Krishna From: Prasanta Sadhukhan Sent: Tuesday, March 20, 2018 5:04 PM To: Krishna Addepalli <krishna.addepa...@oracle.com>; swing-dev@openjdk.java.net Subject: Re: <Swing Dev> [11] RFR JDK-8189687:Swing: Invalid position of candidate pop-up of InputMethod in Hi-DPI on Windows Thanks Krishna for your comment. Modified webrev catering to retrieval of scalefactor of the active monitor being shown http://cr.openjdk.java.net/~psadhukhan/8189687/webrev.01/ Regards Prasanta On 3/20/2018 2:40 PM, Krishna Addepalli wrote: Hi Prasanta, I have couple questions regarding your fix: 1. The AffineTransform object should be retrieved from the Screen on which the window is showing, whereas in your fix, you are directly getting it from the default screen. In multi screen environment, it may not be aligned correctly. 2. Is there any reason to retrieve the object in the top. I mean, the AffineTransform object can be declared inside the “if (haveActiveClient())” block, at the point of use. Thanks, Krishna From: Prasanta Sadhukhan Sent: Tuesday, March 20, 2018 1:01 PM To: HYPERLINK "mailto:swing-dev@openjdk.java.net"swing-dev@openjdk.java.net Subject: <Swing Dev> [11] RFR JDK-8189687:Swing: Invalid position of candidate pop-up of InputMethod in Hi-DPI on Windows Hi All, Please review a fix for an issue where it is seen that for Japanese IME on windows 10 with scaleFactor 1.5, when we enter text using IME popup, the popup is positioned on top of text, thereby obscuring it as seen in this screenshot. Proposed fix is to apply the scaleFactor on the candidate's popup positional coordinates x,y to generate proper coordinates to show this popup webrev: HYPERLINK "http://cr.openjdk.java.net/%7Epsadhukhan/8189687/webrev.00/"http://cr.openjdk.java.net/~psadhukhan/8189687/webrev.00/ The screenshot after the fix is Regards Prasanta
