After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 745551 - Pasting text from gnome-terminal with concatenated whitespace and tabulators, some whitespace occurrences get removed
Pasting text from gnome-terminal with concatenated whitespace and tabulators,...
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Composer
3.15.x (obsolete)
Other Linux
: Normal minor
: ---
Assigned To: Tomas Popela
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2015-03-03 18:42 UTC by joakim.tjernlund
Modified: 2015-03-04 14:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
contents of ttt file (1.22 KB, text/plain)
2015-03-03 18:42 UTC, joakim.tjernlund
Details

Description joakim.tjernlund 2015-03-03 18:42:40 UTC
Created attachment 298466 [details]
contents of ttt file

If I do cat ttt(ttt attached) in a MATE terinal and then paste all of into New Mail window iIwill get the below msg, white space has been trashed.

Using 3.15.91 + 
24 hours        EHTMLEditorView - Fix Paste Quotation action
30 hours        EHTMLEditorView - Selection is lost when showing context menu
30 hours        EHTMLEditorActions - Actions from context menu don't work


# cat ttt
i2c-0   i2cMPC adapter at 0xffe118000I2C adapter
i2c-1   i2cMPC adapter at 0xffe118100I2C adapter
i2c-2   i2cMPC adapter at 0xffe119000I2C adapter
i2c-3   i2cMPC adapter at 0xffe119100I2C adapter
i2c-4   i2ci2c-0-mux (chan_id 0)I2C adapter
i2c-5   i2ci2c-0-mux (chan_id 1)I2C adapter
i2c-6   i2ci2c-0-mux (chan_id 2)I2C adapter
i2c-7   i2ci2c-0-mux (chan_id 3)I2C adapter
i2c-8   i2ci2c-0-mux (chan_id 4)I2C adapter
i2c-9   i2ci2c-0-mux (chan_id 5)I2C adapter
i2c-10  i2ci2c-0-mux (chan_id 6)I2C adapter i2c-11  i2ci2c-0-mux (chan_id 7)I2C adapter i2c-12  i2ci2c-0-mux (chan_id 8)I2C adapter i2c-13  i2ci2c-0-mux (chan_id 9)I2C adapter i2c-14  i2ci2c-0-mux (chan_id 10)I2C adapter i2c-15  i2ci2c-0-mux (chan_id 11)I2C adapter i2c-16  i2ci2c-0-mux (chan_id 12)I2C adapter i2c-17  i2ci2c-0-mux (chan_id 13)I2C adapter i2c-18  i2ci2c-0-mux (chan_id 14)I2C adapter i2c-19  i2ci2c-0-mux (chan_id 15)I2C adapter
Comment 1 joakim.tjernlund 2015-03-03 20:52:42 UTC
I don't think this is minor, consider pasting a patch or something else WS
sensitive. Then the path is trasched and does not apply.
Comment 2 Tomas Popela 2015-03-04 06:44:28 UTC
I can't reproduce it in my setup (xterm + tmux) where is it pasted fine, but it can be reproduced in gnome-terminal
Comment 3 André Klapper 2015-03-04 07:56:18 UTC
(In reply to joakim.tjernlund from comment #1)
> I don't think this is minor, consider pasting a patch or something else WS
> sensitive. Then the path is trasched and does not apply.

If you want patches to be applied, send them as attachments. Whitespace would not be your only problem, there are also random linebreaks and other stuff added on the way.
Comment 4 joakim.tjernlund 2015-03-04 08:11:20 UTC
(In reply to André Klapper from comment #3)
> (In reply to joakim.tjernlund from comment #1)
> > I don't think this is minor, consider pasting a patch or something else WS
> > sensitive. Then the path is trasched and does not apply.
> 
> If you want patches to be applied, send them as attachments. Whitespace
> would not be your only problem, there are also random linebreaks and other
> stuff added on the way.

That is a workaround which you also have to know beforehand.
Plus, try sending patches as attachments to Linux and see what
they think about that.
Comment 5 Tomas Popela 2015-03-04 09:36:50 UTC
Fixed with commit 5b175fa49d2f1bb4f523049120192cf49d238a55 in the master branch for Evolution 3.15.92+.
Comment 6 joakim.tjernlund 2015-03-04 10:16:49 UTC
Nice, works much better!

The TABs are replaced with SPACE though but I think that happens before
Evo gets the data. I guess MATE is resposible.
Comment 7 Tomas Popela 2015-03-04 11:47:02 UTC
(In reply to joakim.tjernlund from comment #6)
> The TABs are replaced with SPACE though but I think that happens before
> Evo gets the data. I guess MATE is resposible.

No Evolution was doing that. But I realized that we can use the WebKit HTML representation on a tabulator and we can simplify that code a little bit.

Improved with commit fd95fbf59de912a63b638e70075cb300f3c6c21b in the master branch for Evolution 3.15.92+.
Comment 8 joakim.tjernlund 2015-03-04 12:01:46 UTC
(In reply to Tomas Popela from comment #7)
> (In reply to joakim.tjernlund from comment #6)
> > The TABs are replaced with SPACE though but I think that happens before
> > Evo gets the data. I guess MATE is resposible.
> 
> No Evolution was doing that. But I realized that we can use the WebKit HTML
> representation on a tabulator and we can simplify that code a little bit.
> 
> Improved with commit fd95fbf59de912a63b638e70075cb300f3c6c21b in the master
> branch for Evolution 3.15.92+.

yes, now it works!

 Thanks
Comment 9 joakim.tjernlund 2015-03-04 12:17:17 UTC
One odd thing, not sure if this is related or not:

Hovering over a web link(https://....com) in a recevied msg I can open that link
in my webrowser but when I hit reply and hovers over the link Evo does not
recognize the link, it is just text.
Comment 10 joakim.tjernlund 2015-03-04 12:48:37 UTC
Another odd thing, this mail(in swedish):
----------
Jag tänker att vi inte behöver släppa detta nu... De kommer ju inte uppgraderas hursom MEN det hade varit bra att:

1. Testa att det kommer att funka den dag det behövs en ny boot/kernel.
2. Checka in ändringarna så det kommer med i nästa bygge vi vill göra av boot/kernel så det följer med per automatik då.
-------------
looks like this in View Msg Source:
-------------
Jag t=E4nker att vi inte beh=F6ver sl=E4ppa detta nu... De kommer ju inte u=
ppgraderas hursom MEN det hade varit bra att:=0A=
=0A=
1. Testa att det kommer att funka den dag det beh=F6vs en ny boot/kernel.=
=0A=
2. Checka in =E4ndringarna s=E5 det kommer med i n=E4sta bygge vi vill g=F6=
ra av boot/kernel s=E5 det f=F6ljer med per automatik d=E5.=0A=
-----------
Why the line breaks/wraps?
Must '\n' be displayed as =0A= ?

Doing reply messes with line breaks:
-----
> Jag tänker att vi inte behöver släppa detta nu... De kommer ju inte uppgraderas hursom MEN det hade varit 
> bra att:
> 
> 1. Testa att det kommer att funka den dag det behövs en ny boot/kernel. 2. Checka in ändringarna så det 
> kommer med i nästa bygge vi vill göra av boot/kernel så det följer med per automatik då.
----
Extra '\n' before "bra att:"
Notice how '\n' is lost just before "2. Checka..."
Comment 11 André Klapper 2015-03-04 13:22:37 UTC
(In reply to joakim.tjernlund from comment #9)
> Hovering over a web link(https://....com) in a recevied msg I can open that
> link in my webrowser

That does not sound whitespace and tabulator related, hence out of scope for this bug report?

> Why the line breaks/wraps? Must '\n' be displayed as =0A= ?

General questions could be asked on the mailing lists: https://mail.gnome.org/mailman/listinfo/evolution-list
Comment 12 joakim.tjernlund 2015-03-04 14:35:10 UTC
(In reply to André Klapper from comment #11)
> (In reply to joakim.tjernlund from comment #9)
> > Hovering over a web link(https://....com) in a recevied msg I can open that
> > link in my webrowser
> 
> That does not sound whitespace and tabulator related, hence out of scope for
> this bug report?

You tell me, I don't know.

> 
> > Why the line breaks/wraps? Must '\n' be displayed as =0A= ?
> 
> General questions could be asked on the mailing lists:
> https://mail.gnome.org/mailman/listinfo/evolution-list

OK, so I mailed every thing to the list. Don't ask me to file a bunch of
new bug reports though.