Bug 155416

Summary:	Colon character next to a forward slash confuses double click select
Product:	[Applications] konsole	Reporter:	Griff Miller <griff.miller>
Component:	general	Assignee:	Konsole Developer <konsole-devel>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	RedHat Enterprise Linux
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description Griff Miller 2008-01-10 20:00:18 UTC

Version:            (using KDE KDE 3.5.4)
Installed from:    RedHat RPMs
OS:                Linux

In:

Settings -> Configure Konsole...
General
Double Click

I have only an underscore, i.e. _ character.

This is because I am used to the way double click selection works in xterm.

Anyway, it works like I expect most of the time, except when : and / are next to each other, in either order.

Look at these strings:

work2:werwe:adwfaser
erwer/werwer/werwer/werwer
dsfasdfasdf/:adsfasdfasdf
sdfasdfasdfasdf:/adfasdfadsf

In the first two examples, double click allows : and / to be highlighted by themselves. In the last two examples, : and / cannot be highlighted individually. Compare to how double click works on those strings in xterm.

In summary, double click is treating :/ and /: as a word, in spite of the fact that I've configured konsole to not treat : and / as part of a word.

Comment 1 Robert Knight 2008-03-30 07:23:05 UTC

SVN commit 791732 by knight:

Fix consecutive non-word characters being treated as a single word.

TerminalDisplay::charClass() used to return one of three values to indicate whether
the input was a space, "word character" (letter, number or character from the wordCharacters() array)
or something else.  When double clicking a 'word' was taken to be a run of
characters which had the same charClass().  Fix this by returning the input character
in charClass() if it is not a space or word character.

CCBUG:155416


 M  +6 -8      TerminalDisplay.cpp  
 M  +7 -1      TerminalDisplay.h  


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

Comment 2 Robert Knight 2008-05-23 22:27:03 UTC

Marking as fixed following comment #1