[Webkit-unassigned] [Bug 19066] ASSERT in editing code, ASSERTION FAILED: isStartOfParagraph(startOfParagraphToMove)

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Mar 9 14:18:09 PDT 2009


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





------- Comment #8 from eric at webkit.org  2009-03-09 14:18 PDT -------
(In reply to comment #7)
> This code wants to get the start of the paragraph so that it moves the whole
> paragraph into the new list item.  I'm not sure why it's jumping from [img, 0]
> to [body, 0], though.

This jump kinda makes sense.  It's jumping to the containing block (which is
the body), and setting start to the first offset in the containing block (which
makes sense).

I'm not sure that it makes sense though to insert the list right before the
image, and then move the image inside it.  Or at least, by doing so, we change
what the "start of the paragraph" is, since now the paragraph should start
right after the list.

Maybe the right behavior here would be to first wrap the content which we
indend to move into the list, into a new block.  Then insert the list before
the block.  And then move the contents of the block into the list, and remove
the fake block.

Alternatively, we could just learn to update the start position of the
paragraph after we insert the list. :)  I'd be interested to hear your thoughts
Justin.


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



More information about the webkit-unassigned mailing list