57170 - copy/paste text does not carry text attribute

Status: VERIFIED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[Frank Tang]

I observer this behavior by using my 10/16 branch pull build on window. I think
it is very hard to use editor without fix this issue.

reproduce procedure:
1. open composer
2. type abcdefghijklmnopqrstuv
3. select abcdefghijklmnopqrstuv
4. make the color to red by select "Format:Text color..." and pick a color (say
red) and click OK
5. now abcdefghijklmnopqrstuv is red, which is ok.
6. select cdefg
7. copy
8. move cursor after s
9. paste
expect result
cdefg paste after s with color red
actual result
cdefg paste after s with color black.

The interesting thing is if you copy the whole text "abcdefghijklmnopqrstuv
" and paste, then it will paste as in red.

Comment 1

[Frank Tang]

This not only apply to text color, but text size.
Text font is also broken.
<b> <I> and <U> is ok but other text style are broken.

Comment 2

[rubydoo123]

this is by design, if you select the entire element content, then you get the
content and the element, if you make a partial selection, then you get just the
content. Joe has been working on changing this model for some selection points,
so I am reassigning this to hime for 1. clarification, 2. disposition

Comment 3

[Joe Francis]

I could change copy to grab any inline style, not just {b,i,u}.  That would be 
fine with me.  I don't know why we decided to use only {b,i,u}.

Comment 4

[Joe Francis]

moz 0.9

Comment 5

[rubydoo123]

setting to rtm -

Comment 6

[Joe Francis]

Attachment: patch to a couple of layout files.

Comment 7

[Joe Francis]

patch attached.  need review/sr.  akkana? simon?

Comment 8

[Akkana Peck]

Is it safe to include td and th in IsRoot?  Should we possibly rename IsRoot if
it means something else?  Looks fine aside from that question.

Comment 9

[Simon Fraser [no longer active]]

I think we need to think more about copying block-level elements from the parent 
hierarchy, but those were there already. sr=sfraser

Comment 10

[Joe Francis]

fixed

Comment 11

[James Baxter]

Verified on 8-28 build