Fix for setSelection( htmlDocument().createRange() )

Leo Savernik l.savernik at aon.at
Fri Nov 5 16:04:00 GMT 2004


Am Freitag, 5. November 2004 16:19 schrieb David Faure:
> > In this case, RenderCanvas::setSelection will select the whole div.
>
> Ouch - this is wrong, isn't it?
> The right fix is in setSelection then, right? Any chance you could have a
> look? :/

Factually, the representation of text selection boundaries and Ranges are 
incompatible, and cannot be solved quickly (WebCore has some "cheats", but 
they will take me a while). As collapsed ranges should be common, I consider 
your patch with Marc's improvement a good quick-fix (well-suited for 
backporting).

mfg
 Leo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20041105/eb7023c9/attachment.sig>


More information about the kfm-devel mailing list