eirikbakke commented on code in PR #8310:
URL: https://github.com/apache/netbeans/pull/8310#discussion_r1990433451


##########
platform/core.windows/src/org/netbeans/core/windows/view/dnd/DragWindow.java:
##########
@@ -95,15 +102,17 @@ private BufferedImage createContentImage( Component c, 
Dimension contentSize ) {
         GraphicsConfiguration config = 
GraphicsEnvironment.getLocalGraphicsEnvironment()
                     .getDefaultScreenDevice().getDefaultConfiguration();
 
-        BufferedImage res = config.createCompatibleImage(contentSize.width, 
contentSize.height);
+        BufferedImage res = config.createCompatibleImage(
+                contentSize.width * DPI_SCALE, contentSize.height * DPI_SCALE);
         Graphics2D g = res.createGraphics();
+        g.scale(DPI_SCALE, DPI_SCALE);

Review Comment:
   In the default configuration, it's a small thumbnail rectangle as seen in 
the screenshot in the PR, not a full-size copy of the component.
   
   But looking at DragAndDropFeedbackVisualizer, which instantiates the 
DragWindow, there is a WinSysPrefs.DND_SMALLWINDOWS option that can be turned 
off to get the full-size window. If a platform app happens to be overriding 
that setting, it's true that the buffer could be several megabytes.
   
   Should I make some efforts to skip the scaling in the 
WinSysPrefs.DND_SMALLWINDOWS=false case?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org
For additional commands, e-mail: notifications-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to