When I drag a link from a 2x display to a 1x display, the text in the drag image is blurry.
<rdar://problem/33519698>
Created attachment 316388 [details] Patch
Comment on attachment 316388 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=316388&action=review > Source/WebCore/platform/mac/DragImageMac.mm:293 > + // To work around blurry drag images on 1x displays, make the width and height a multiple of 2. > + // FIXME: remove this workaround when <rdar://problem/33059739> is fixed. > + boundingRect.setWidth((static_cast<int>(boundingRect.width()) / 2) * 2); > + boundingRect.setHeight((static_cast<int>(boundingRect.height() / 2) * 2)); Is this a user-visible change, or does it just add transparent pixels? Does it affect alignment of the image with the pointer?
(In reply to Simon Fraser (smfr) from comment #3) > Comment on attachment 316388 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=316388&action=review > > > Source/WebCore/platform/mac/DragImageMac.mm:293 > > + // To work around blurry drag images on 1x displays, make the width and height a multiple of 2. > > + // FIXME: remove this workaround when <rdar://problem/33059739> is fixed. > > + boundingRect.setWidth((static_cast<int>(boundingRect.width()) / 2) * 2); > > + boundingRect.setHeight((static_cast<int>(boundingRect.height() / 2) * 2)); > > Is this a user-visible change, or does it just add transparent pixels? Does > it affect alignment of the image with the pointer? Do you mean if this changes the alignment of the memory address of the image data? Thanks for reviewing!
Comment on attachment 316388 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=316388&action=review >>> Source/WebCore/platform/mac/DragImageMac.mm:293 >>> + boundingRect.setHeight((static_cast<int>(boundingRect.height() / 2) * 2)); >> >> Is this a user-visible change, or does it just add transparent pixels? Does it affect alignment of the image with the pointer? > > Do you mean if this changes the alignment of the memory address of the image data? > > Thanks for reviewing! No, i mean the position of the drag image relative to the mouse pointer.
(In reply to Simon Fraser (smfr) from comment #5) > Comment on attachment 316388 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=316388&action=review > > >>> Source/WebCore/platform/mac/DragImageMac.mm:293 > >>> + boundingRect.setHeight((static_cast<int>(boundingRect.height() / 2) * 2)); > >> > >> Is this a user-visible change, or does it just add transparent pixels? Does it affect alignment of the image with the pointer? > > > > Do you mean if this changes the alignment of the memory address of the image data? > > > > Thanks for reviewing! > > No, i mean the position of the drag image relative to the mouse pointer. Ah, yes, I believe it does affect the alignment with the pointer, since the image size is used in this calculation.
That's bad, then.
Thanks for reviewing!
Comment on attachment 316388 [details] Patch Clearing flags on attachment: 316388 Committed r219891: <http://trac.webkit.org/changeset/219891>
All reviewed patches have been landed. Closing bug.