[Webkit-unassigned] [Bug 27522] Creating list clears background color

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=27522





--- Comment #12 from Ryosuke Niwa <rniwa at webkit.org>  2010-07-13 23:40:59 PST ---
(In reply to comment #11)
> (In reply to comment #10)
> > Created an attachment (id=47180)
 --> (https://bugs.webkit.org/attachment.cgi?id=47180) [details] [details]
> > work in progress
> 
> I'm not done with this, but I wanted to post it to see if it's moving in the right direction.  There are some issues with an extra <br> getting inserted by moveParagraphWithClones and I need to make a thorough layout test for this.

Will your extra BRs go away if you used
    if (beforeParagraph.isNotNull() && !isTableElement(beforeParagraph.deepEquivalent().node()) && ((!isEndOfParagraph(beforeParagraph) && !isStartOfParagraph(beforeParagraph)) || beforeParagraph == afterParagraph)) {

instead of
    if (beforeParagraph.isNotNull() && !isTableElement(beforeParagraph.deepEquivalent().node()) && (!isEndOfParagraph(beforeParagraph) || beforeParagraph == afterParagraph)) {

for the last if statement in moveParagraphWithClones?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.