Bug 153375

Summary: [PATCH] Select All does not select all
Product: [Applications] kate Reporter: Edwin Schepers <yez>
Component: kwriteAssignee: 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:
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