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