[Bug 57364] grep view doesn´t go the correct line

[Joachim Eibl]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
     
http://bugs.kde.org/show_bug.cgi?id=57364     
joachim.eibl at gmx.de changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |joachim.eibl at gmx.de



------- Additional Comments From joachim.eibl at gmx.de  2003-04-21 18:05 -------
Hi, 
 
I use kdevelop 3.0a4a and have a problem with the grep-function: 
 
When I scroll down in the grep-results, and select a result-line then the 
editor shows a wrong position. This doesn't happen, if I don't scroll down. 
Another precondition is that I have selected "Devices -> Mouse -> 
Double click to open files/directories" in the control-center. When I use 
"Single click to open files/directories" everything works fine. 
 
I've analyzed the problem a bit more and the problem seems to be the 
KListBox-implementation, which doesn't behave correctly for "Double-clickers". 
 
Nevertheless I would recommend to change the GrepViewWidget so that it always 
connects the signal clicked() instead of executed(). 
 
kdevelop-3.0a4a/parts/grepview/grepviewwidget.cpp 
... 
GrepViewWidget::GrepViewWidget(GrepViewPart *part) 
    : ProcessWidget(0, "grep widget") 
    , m_matchCount(0) 
{ 
    grepdlg = new GrepDialog(this, "grep widget"); 
    connect( grepdlg, SIGNAL(searchClicked()), 
             this, SLOT(searchActivated()) ); 
    connect( this, SIGNAL(clicked(QListBoxItem*)),  // clicked instead of executed 
             this, SLOT(slotExecuted(QListBoxItem*)) ); 
 
    m_part = part; 
} 
... 
 
Best regards, 
Joachim 
joachim.eibl at gmx.de 
 
PS: Otherwise I use Suse 8.2, with KDE 3.1.1.