Bug 155416 - Colon character next to a forward slash confuses double click select
Summary: Colon character next to a forward slash confuses double click select
Status: RESOLVED FIXED
Alias: None
Product: konsole
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: RedHat Enterprise Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Konsole Developer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-10 20:00 UTC by Griff Miller
Modified: 2008-05-23 22:27 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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