Bug 153375

Summary:	[PATCH] Select All does not select all
Product:	[Applications] kate	Reporter:	Edwin Schepers <yez>
Component:	kwrite	Assignee:	KWrite Developers <kwrite-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	patch for review

Description Edwin Schepers 2007-12-03 22:06:40 UTC

Version:           latest svn (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc
OS:                Linux (x86_64) release 2.6.22-5-default

Hi,
When I ctrl-A on this document ( view source in konqueror on http://doc.trolltech.com/4.3-snapshot/qfiledialog.html )
and then ctrl-A, only some text is selected.
When I put focus on another window or resize, only then all text gets selected.

Regards,
Edwin

Comment 1 Thomas Friedrichsmeier 2007-12-05 16:20:25 UTC

Created attachment 22357 [details]
patch for review

The attached patch appears to fix the problem, but please review, and tell me
whether it's ok to commit.

Problem seems to be that KateViewInternal::tagLines() (called from
KateView::setSelection()) never updates the view cache. While at it, I removed
a bit of redundant logic in KateViewInternal::updateSelection(). This calls
KateView::clearSelection(), which is (now) enough to update the view.

Comment 2 Hamish Rodda 2007-12-06 05:50:08 UTC

SVN commit 745398 by rodda:

Commit slightly modified version of patch provided by Thomas Friedrichsmeier to fix bug 153375 (select all not correctly rendered)

The only modification was to place the update call after the sanity checks, if the tagging was not for lines on the view, we don't need to update at this point.

BTW, much of this will have to be re-done post 4.0 in order to fix an entirely separate issue (memory usage terrible for large files without newlines)

BUG:153375


 M  +2 -6      kateviewinternal.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=745398