[webkit-reviews] review granted: [Bug 24527] caret does not paint the first time you focus on a 0px right padding RTL div : [Attachment 29321] patch w/ Layout test (version 2)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 7 17:59:01 PDT 2009


Simon Fraser (smfr) <simon.fraser at apple.com> has granted Xiaomei Ji
<xji at chromium.org>'s request for review:
Bug 24527: caret does not paint the first time you focus on a 0px right padding
RTL div
https://bugs.webkit.org/show_bug.cgi?id=24527

Attachment 29321: patch w/ Layout test (version 2)
https://bugs.webkit.org/attachment.cgi?id=29321&action=review

------- Additional Comments from Simon Fraser (smfr) <simon.fraser at apple.com>

> Index: LayoutTests/editing/selection/caret-rtl-3.html
> ===================================================================
> --- LayoutTests/editing/selection/caret-rtl-3.html	(revision 0)
> +++ LayoutTests/editing/selection/caret-rtl-3.html	(revision 0)
> @@ -0,0 +1,35 @@
> +<html>
> +<head>
> +<script>
> +if (window.layoutTestController)
> +	layoutTestController.dumpEditingCallbacks();
> +</script>
> +
> +<script>

No need to close and re-open the script tag.

If you wanted the pixel test to make the result more obvious, you could make
the text really big (font-size: 400%) so the caret is larger.


More information about the webkit-reviews mailing list