Bug 54353 - The update area of CSS-transformed with contenteditable does not take the focus ring into account, leading to visual artefacts
Summary: The update area of CSS-transformed with contenteditable does not take the foc...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-13 04:59 PST by Benjamin Poulain
Modified: 2011-03-05 08:48 PST (History)
3 users (show)

See Also:


Attachments
Test case (467 bytes, text/html)
2011-02-13 04:59 PST, Benjamin Poulain
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Poulain 2011-02-13 04:59:41 PST
Created attachment 82260 [details]
Test case

Example attached, we do not update a large enough area for the previous position.

My bet is on a round() of the scaled coordinates instead of taking the biggest rect, but I'll see on debug :)
Comment 1 Alexey Proskuryakov 2011-02-13 11:45:40 PST
Cool effect!
Comment 2 mitz 2011-02-13 11:48:31 PST
I think it’s just that the focus ring width is unaffected by the 2D transform, but the invalidation math assumes that it is.
Comment 3 Benjamin Poulain 2011-02-24 09:10:47 PST
(In reply to comment #2)
> I think it’s just that the focus ring width is unaffected by the 2D transform, but the invalidation math assumes that it is.

Yep, you are right.
Is that the intended behavior to not transform the focus ring? It seems a bit weird.

Should I look for a solution or do you have the intention to change this behavior? :)
Comment 4 mitz 2011-02-25 11:16:00 PST
(In reply to comment #3)
> (In reply to comment #2)
> > I think it’s just that the focus ring width is unaffected by the 2D transform, but the invalidation math assumes that it is.
> 
> Yep, you are right.
> Is that the intended behavior to not transform the focus ring? It seems a bit weird.

I don’t think this was intentional.

> Should I look for a solution or do you have the intention to change this behavior? :)

I think a good solution would be to have the focus ring respect the transform. I am not sure how easy or even feasible this is.
Comment 5 Benjamin Poulain 2011-03-05 08:48:59 PST
Reset the assignee. I wished I would have time to fix this but I looks like I won't in the near future.