Summary: | Text transformed with text-transform does not copy with transform applied | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andrew Wellington <andrew> | ||||||
Component: | HTML Editing | Assignee: | Darin Adler <darin> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ap, nickshanks | ||||||
Priority: | P2 | ||||||||
Version: | 412 | ||||||||
Hardware: | Mac | ||||||||
OS: | OS X 10.4 | ||||||||
Attachments: |
|
Description
Andrew Wellington
2005-06-10 23:20:14 PDT
This has been fixed in Konqueror *** Bug 4427 has been marked as a duplicate of this bug. *** TextIterator is the thing that would need to be changed, in visible_text.h/cpp. Created attachment 3378 [details]
Patch to use renderer string instead of DOM string
Modify visible_text.cpp to use the string from the renderer not the DOM in
handleTextNode and handleTextBox
Comment on attachment 3378 [details]
Patch to use renderer string instead of DOM string
This looks like the right fix. But we also need a test to check in at the same
time. I'm going to set this to review-, but it should be set to review+ once we
have a suitable test.
Two ways to write the test: 1) Use the "innerText" method. 2) Use execCommand("PasteAndMatchStyle") to make a test that will work in DumpRenderTree, although not in Safari. (Because the innerText method shares the same code path with copy.) Created attachment 3391 [details]
Regression Test
Proposed regression test
|