Bug 64537 - tab character not converted to spaces; word wrapping breaks.
Summary: tab character not converted to spaces; word wrapping breaks.
Status: RESOLVED UPSTREAM
Alias: None
Product: kmail
Classification: Applications
Component: composer (show other bugs)
Version: 1.10.0
Platform: Debian testing Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords: testcase, triaged
: 79737 151358 (view as bug list)
Depends on:
Blocks:
 
Reported: 2003-09-19 11:33 UTC by Amit Shah
Modified: 2009-08-26 21:06 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Testcase with broken wrapping (3.05 KB, application/mbox)
2008-08-31 23:52 UTC, Dominik Tritscher
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Amit Shah 2003-09-19 11:33:09 UTC
Version:           1.5.3 (using KDE KDE 3.1.3)
Installed from:    Debian testing/unstable Packages
OS:          Linux

The wordwrapping breaks in the composer window if tabs are included in a line. It is recognized as one character, so the wordwrapping algorithm increments the column count by just one, where infact, it should be incremented by 8 (tab size).
Comment 1 Tom Albers 2004-08-07 14:01:36 UTC
I can confirm this bug still exists in the development version of KMail. Just paste the test 'wrap test ' 30 times so you see where the wrapping supposed to be and then press tab and paste a couple of times extra to see the word wrap for that line. You see this is not working as it should.
Comment 2 Tom Albers 2004-08-07 23:29:01 UTC
*** Bug 79737 has been marked as a duplicate of this bug. ***
Comment 3 Daniel Phillips 2004-11-08 05:20:59 UTC
Kmail composer replaces tabs by single spaces before sending or saving in drafts.

I can confirm that this bug exists in Kmail 1.7 under Gentoo and Debian Sid, and in the 1.7.50 nightly snapshot, which I have built for debugging.  This bug did not exist in Kmail 1.6.2 or earlier.
Comment 4 Daniel Phillips 2004-11-08 05:32:28 UTC
Steps to demonstrate this bug:

  - Compose a new mail

  - Enter text:
      <tab>sometext

  - Save in drafts

  - Open mail in drafts, the text of the mail is:
      <space>sometext

  - Save the mail to a file, this demonstrates that the tab
    has really changed to a space, not just been incorrectly
    displayed
Comment 5 Daniel Phillips 2004-11-08 08:29:55 UTC
I admit it, I'm a c++ klutz, however I think I've gone about as far as I can go
without building kdelibs and qt for debug also, which I don't want to do on my
workstation, maybe I'll try it later on the Gentoo box.

The badness has already happened in MessageComposer::breakLinesAndApplyCodec()
at text = mComposeWin->mEditor->text():

   (gdb) p text.ascii() 
   $23 = 0x8341500 " text"
   # this should have been "\ttext" 

which suggests the problem is deep in KEdit.  If I poke a tab into the string,
and continue, I get a complaint later that "Not all characters fit into the 
chosen encoding."  I chose "send anyway (and lose characters)", and lo, the tab 
made it through to the drafts folder.  The encoding is supposedly set to 
us-ascii.  Last time I checked, us-ascii had a tab.  Maybe this is a clue.
Comment 6 Luke-Jr 2004-12-18 02:59:58 UTC
This bug report is a bug in itself-- I want my tabs to stay tabs, with a visible width of 4.
Comment 7 Edwin Schepers 2004-12-18 18:33:31 UTC
it looks like the original report doesn't speak about the bug that a tab is converted to a space, but about a bug in the wordwrapping.
About the bug about tabs being converted to spaces, I refer to 90688
Comment 8 Thomas McGuire 2007-11-02 17:30:45 UTC
*** Bug 151358 has been marked as a duplicate of this bug. ***
Comment 9 Dominik Tritscher 2008-08-31 23:52:11 UTC
Created attachment 27153 [details]
Testcase with broken wrapping

I created a simple testcase containing several lines with correct wrapping. In the first line i added some tabs to show that the line gets longer as it should.
With that testcase I can confirm that problem using KMail 1.10.0
Comment 10 Martin Koller 2009-08-26 21:06:46 UTC
Still the case with Qt-4.5.2/KDE 4.3
This is a Qt bug. It wraps lines containing Tabs not at the defined FixedColumnWidth.

I reported the bug to Qt-Software