Bug 34914 – copy-pasted text with font-size and full-page-zoom gets too large

• Component: HTML Editing
• Status: RESOLVED FIXED

[Ojan Vafai]

Created attachment 48683 [details]
testcase

See the test case. Also, chromium bug with repro inside of gmail: http://code.google.com/p/chromium/issues/detail?id=22804

[Tony Chang]

(In reply to comment #0)
> Created an attachment (id=48683) [details]
> testcase
> 
> See the test case. Also, chromium bug with repro inside of gmail:
> http://code.google.com/p/chromium/issues/detail?id=22804

There are some differences here with Firefox.  Here's an example using font-size rather than zoom:
http://www.plexode.com/cgi-bin/eval3.py#ht=%3Cdiv%20contenteditable%3Dtrue%3E%3Cdiv%20style%3D%22font-size%3A%20200%25%22%3Eheading%3C%2Fdiv%3E%3Cdiv%3Eregular%20text%3C%2Fdiv%3E%0A%3Cdiv%3Emore%20text%3C%2Fdiv%3E%0A%3C%2Fdiv%3E&ohh=1&ohj=1&jt=&ojh=1&ojj=1&ms=100&oth=0&otj=0&cex=1

In Firefox, if you copy a few letters from heading and paste it into the lines below it, it matches the font size of where you're pasting into.  In WebKit, it matches the font size of the copied text.  WebKit seems to match TextEdit here.

I guess in this case, we want the pasted text to be the exact same size as what we copied.  This would be consistent with what WebKit does in the case of the different font-sizes.

[Tony Chang]

Created attachment 48741 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 48741 [details]
Patch

LGTM.

[WebKit Commit Bot]

Comment on attachment 48741 [details]
Patch

Clearing flags on attachment: 48741

Committed r54932: <http://trac.webkit.org/changeset/54932>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Tony Chang]

Committed r54935: <http://trac.webkit.org/changeset/54935>