210935 2020-04-23 14:36:30 -0700 Resize cursor doesn't show if there's text under the resize control 2025-05-14 09:27:28 -0700 1 1 1 Unclassified WebKit UI Events Safari Technology Preview Unspecified Unspecified RESOLVED FIXED https://bugs.webkit.org/show_bug.cgi?id=156234 BrowserCompat, InRadar P2 Normal --- 1 simon.fraser webkit-unassigned ahmad.saleem792 akeerthi rniwa simon.fraser webkit-bug-importer oldest_to_newest 1644966 0 397385 simon.fraser 2020-04-23 14:36:30 -0700 Created attachment 397385 Test Hover over the resize cursor in the attached testcase. You don't see the resize cursor, instead getting an iBeam. 1909170 1 ahmad.saleem792 2022-10-29 11:24:43 -0700 It seems to be fixed with following Commit: https://src.chromium.org/viewvc/blink?view=revision&revision=160554 It did changes here: https://github.com/WebKit/WebKit/blob/9856bc8b756610ac4e3a4a2c480e6332f50e0cd7/Source/WebCore/page/EventHandler.cpp#L1544 ___ Webkit (current): inResizer = layerRenderer.layer()->isPointInResizeControl(roundedIntPoint(result.localPoint())); Chrome Patch: inResizer = layer->isPointInResizeControl(result.roundedPointInMainFrame(), ResizerForPointer); *** Need to be changed *** inResizer = layerRenderer.layer()->isPointInResizeControl(roundedIntPoint(result.roundedPointInMainFrame(), localPoint); ___ If someone can confirm the approach, I am happy to do PR. Only problem / build error might be that it could only accept 1 argument but now we are providing it two. For that, if we can do: auto joined = roundedIntPoint(result.roundedPointInMainFrame(), localPoint); and then pass on single argument. 1909171 2 ahmad.saleem792 2022-10-29 11:25:35 -0700 To this: inResizer = layerRenderer.layer()->isPointInResizeControl(result.roundedPointInMainFrame(), localPoint); 2115977 3 alexbdamac 2025-05-08 15:53:31 -0700 Pull request: https://github.com/WebKit/WebKit/pull/45131 2117076 4 ews-feeder 2025-05-14 09:23:32 -0700 Committed 294907@main (342a27a71e7c): <https://commits.webkit.org/294907@main> Reviewed commits have been landed. Closing PR #45131 and removing active labels. 2117078 5 webkit-bug-importer 2025-05-14 09:27:28 -0700 <rdar://problem/151309503> 397385 2020-04-23 14:36:30 -0700 2020-04-23 14:36:30 -0700 Test mouse-cursor-over-resizer.html text/html 3282 simon.fraser PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KICAgIDxzdHlsZT4KICAgICAgICAjdGFyZ2V0 IHsKICAgICAgICAgICAgb3ZlcmZsb3c6IGhpZGRlbjsKICAgICAgICAgICAgd2lkdGg6IDIwMHB4 OwogICAgICAgICAgICBoZWlnaHQ6IDEwMHB4OwogICAgICAgICAgICB0ZXh0LWFsaWduOiByaWdo dDsKICAgICAgICAgICAgY3Vyc29yOiBhdXRvICFpbXBvcnRhbnQ7CiAgICAgICAgICAgIHJlc2l6 ZTogYm90aDsKICAgICAgICAgICAgYm9yZGVyOiAxMnB4IHNvbGlkIHNpbHZlcjsKICAgICAgICB9 CiAgICA8L3N0eWxlPgogICAgPHNjcmlwdCBzcmM9Ii4uLy4uL3Jlc291cmNlcy9qcy10ZXN0LXBy ZS5qcyI+PC9zY3JpcHQ+CiAgICA8c2NyaXB0PgogICAgICAgIHdpbmRvdy5qc1Rlc3RJc0FzeW5j ID0gdHJ1ZTsKCiAgICAgICAgdmFyIENVUlNPUl9VUERBVEVfREVMQVkgPSAyMDA7CgogICAgICAg IGZ1bmN0aW9uIHJ1blRlc3QocHJlcGFyZSwgbmV4dCkgewogICAgICAgICAgICBwcmVwYXJlKCk7 CiAgICAgICAgICAgIHNldFRpbWVvdXQoZnVuY3Rpb24oKSB7CiAgICAgICAgICAgICAgICBkZWJ1 ZygnQ3Vyc29yIEluZm86ICcgKyB3aW5kb3cuaW50ZXJuYWxzLmdldEN1cnJlbnRDdXJzb3JJbmZv KCkpOwogICAgICAgICAgICAgICAgZGVidWcoJycpOwogICAgICAgICAgICAgICAgbmV4dCgpOwog ICAgICAgICAgICB9LCBDVVJTT1JfVVBEQVRFX0RFTEFZKTsKICAgICAgICB9CgogICAgICAgIGZ1 bmN0aW9uIHRlc3RzRG9uZSgpIHsKICAgICAgICAgICAgLy8gVGhpcyB0ZXh0IGlzIHJlZHVuZGFu dCB3aXRoIHRoZSB0ZXN0IG91dHB1dCAtIGhpZGUgaXQKICAgICAgICAgICAgLy8gZG9jdW1lbnQu Z2V0RWxlbWVudEJ5SWQoJ3Rlc3QtY29udGFpbmVyJykuc3R5bGUuZGlzcGxheSA9ICdub25lJzsK ICAgICAgICAgICAgZmluaXNoSlNUZXN0KCk7CiAgICAgICAgfQoKICAgICAgICBmdW5jdGlvbiBz ZXR1cFRlc3QoKQogICAgICAgIHsKICAgICAgICAgICAgZGVzY3JpcHRpb24oIlRlc3RzIHRoYXQg dGhlIG1vdXNlIGN1cnNvciB1cGRhdGVzIG92ZXIgdGhlIHJlc2l6ZSBjb250cm9sLiIpOwoKICAg ICAgICAgICAgaWYgKCF3aW5kb3cuZXZlbnRTZW5kZXIpIHsKICAgICAgICAgICAgICAgIHRlc3RG YWlsZWQoJ1RoaXMgdGVzdCByZXF1aXJlcyBEdW1wUmVuZGVyVHJlZScpOwogICAgICAgICAgICAg ICAgcmV0dXJuOwogICAgICAgICAgICB9CgogICAgICAgICAgICAvLyBDYW4ndCBkbyBhbnl0aGlu ZyB1c2VmdWwgaGVyZSB3aXRob3V0IGV2ZW50U2VuZGVyCiAgICAgICAgICAgIGlmICh3aW5kb3cu ZXZlbnRTZW5kZXIpIHsKICAgICAgICAgICAgICAgIHZhciB0YXJnZXQgPSBkb2N1bWVudC5nZXRF bGVtZW50QnlJZCgndGFyZ2V0Jyk7CiAgICAgICAgICAgICAgICBldmVudFNlbmRlci5kcmFnTW9k ZSA9IGZhbHNlOwogICAgICAgICAgICAgICAgdmFyIHRlc3RzID0gWwogICAgICAgICAgICAgICAg ICAgIGZ1bmN0aW9uKCkgewogICAgICAgICAgICAgICAgICAgICAgICBkZWJ1ZygnTW91c2UgbW92 ZScpOwogICAgICAgICAgICAgICAgICAgICAgICBldmVudFNlbmRlci5tb3VzZU1vdmVUbyh0YXJn ZXQub2Zmc2V0TGVmdCArIDMsIHRhcmdldC5vZmZzZXRUb3AgKyAzKTsKICAgICAgICAgICAgICAg ICAgICB9LAogICAgICAgICAgICAgICAgICAgIGZ1bmN0aW9uKCkgewogICAgICAgICAgICAgICAg ICAgICAgICBkZWJ1ZygnTW91c2UgZG93bicpOwogICAgICAgICAgICAgICAgICAgICAgICBldmVu dFNlbmRlci5tb3VzZURvd24oKTsKICAgICAgICAgICAgICAgICAgICB9LAogICAgICAgICAgICAg ICAgICAgIGZ1bmN0aW9uKCkgewogICAgICAgICAgICAgICAgICAgICAgICBkZWJ1ZygnTW91c2Ug aG9sZCBkb3duLCBtb3ZlJyk7CiAgICAgICAgICAgICAgICAgICAgICAgIGV2ZW50U2VuZGVyLm1v dXNlTW92ZVRvKHRhcmdldC5vZmZzZXRMZWZ0ICsgMTMsIHRhcmdldC5vZmZzZXRUb3AgKyAzKTsK ICAgICAgICAgICAgICAgICAgICB9LAogICAgICAgICAgICAgICAgICAgIGZ1bmN0aW9uKCkgewog ICAgICAgICAgICAgICAgICAgICAgICBkZWJ1ZygnTW91c2UgdXAnKTsKICAgICAgICAgICAgICAg ICAgICAgICAgZXZlbnRTZW5kZXIubW91c2VVcCgpOwogICAgICAgICAgICAgICAgICAgIH0KICAg ICAgICAgICAgICAgIF07CgogICAgICAgICAgICAgICAgdmFyIGkgPSAwOwoKICAgICAgICAgICAg ICAgIGZ1bmN0aW9uIG5leHRUZXN0KCkgewogICAgICAgICAgICAgICAgICAgIGlmIChpIDwgdGVz dHMubGVuZ3RoKSB7CiAgICAgICAgICAgICAgICAgICAgICAgIHJ1blRlc3QodGVzdHNbaSsrXSwg bmV4dFRlc3QpOwogICAgICAgICAgICAgICAgICAgIH0gZWxzZSB7CiAgICAgICAgICAgICAgICAg ICAgICAgIHRlc3RzRG9uZSgpOwogICAgICAgICAgICAgICAgICAgIH0KICAgICAgICAgICAgICAg IH0KICAgICAgICAgICAgICAgIG5leHRUZXN0KCk7CiAgICAgICAgICAgIH0KICAgICAgICB9CiAg ICAgICAgCiAgICAgICAgd2luZG93LmFkZEV2ZW50TGlzdGVuZXIoJ2xvYWQnLCAoKSA9PiB7CiAg ICAgICAgICAgIHNldHVwVGVzdCgpOwogICAgICAgIH0sIGZhbHNlKTsKICAgIDwvc2NyaXB0Pgo8 L2hlYWQ+Cjxib2R5PgogICAgPGRpdiBpZD0idGVzdC1jb250YWluZXIiPgogICAgICAgIDxkaXYg aWQ9InRhcmdldCI+CiAgICAgICAgICAgIExvcmVtIGlwc3VtIGRvbG9yIHNpdCBhbWV0LCBjb25z ZWN0ZXR1ciBhZGlwaXNpY2luZyBlbGl0LCBzZWQgZG8gZWl1c21vZCB0ZW1wb3IgaW5jaWRpZHVu dCB1dCBsYWJvcmUgZXQgZG9sb3JlIG1hZ25hIGFsaXF1YS4gVXQgZW5pbSBhZCBtaW5pbSB2ZW5p YW0sIHF1aXMgbm9zdHJ1ZCBleGVyY2l0YXRpb24gdWxsYW1jbyBsYWJvcmlzIG5pc2kgdXQgYWxp cXVpcCBleCBlYSBjb21tb2RvIGNvbnNlcXVhdC4gRHVpcyBhdXRlIGlydXJlIGRvbG9yIGluIHJl cHJlaGVuZGVyaXQgaW4gdm9sdXB0YXRlIHZlbGl0IGVzc2UgY2lsbHVtIGRvbG9yZSBldSBmdWdp YXQgbnVsbGEgcGFyaWF0dXIuIEV4Y2VwdGV1ciBzaW50IG9jY2FlY2F0IGN1cGlkYXRhdCBub24g cHJvaWRlbnQsIHN1bnQgaW4gY3VscGEgcXVpIG9mZmljaWEgZGVzZXJ1bnQgbW9sbGl0IGFuaW0g aWQgZXN0IGxhYm9ydW0uCiAgICAgICAgPC9kaXY+CiAgICA8L2Rpdj4KICAgIDxkaXYgaWQ9ImNv bnNvbGUiPjwvZGl2PgogICAgPHNjcmlwdCBzcmM9Ii4uLy4uL3Jlc291cmNlcy9qcy10ZXN0LXBv c3QuanMiPjwvc2NyaXB0Pgo8L2JvZHk+CjwvaHRtbD4K