Fix for setSelection( htmlDocument().createRange() )
David Faure
faure at kde.org
Fri Nov 5 15:19:19 GMT 2004
On Friday 05 November 2004 16:20, Leo Savernik wrote:
> Am Freitag, 5. November 2004 12:35 schrieb David Faure:
> > I guess you meant you were against the change Marc suggested, but not
> > against mine?
>
> I only wanted to annotate that Marc's suggestion is more general. I. e. did
> you check what happens on the snippet
>
> <html>
> <div>bla</div>
> </html>
>
> when the Range is (<div>, 0)?
>
> 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? :/
--
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
More information about the kfm-devel
mailing list