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 741402 - vteapp iconified by screen
vteapp iconified by screen
Status: RESOLVED FIXED
Product: vte
Classification: Core
Component: general
0.39.x
Other Linux
: Normal normal
: ---
Assigned To: VTE Maintainers
VTE Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-12-11 16:24 UTC by Egmont Koblinger
Modified: 2014-12-11 19:24 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Fix (928 bytes, patch)
2014-12-11 16:35 UTC, Egmont Koblinger 		none Details | Review
Fix v2 (16.41 KB, patch)
2014-12-11 17:04 UTC, Egmont Koblinger 		committed Details | Review

Description Egmont Koblinger 2014-12-11 16:24:10 UTC 
Start vteapp, inside that start screen (or attach to / detach from a session, or leave screen).

Vteapp window is iconified.

Addording to the vte/src/window script, \e[2t should iconify the window. That's what xterm does too, and so does vteapp.

screen emits \e[22;2t and \e[23;2t. We should understand what these are, and know that these are not meant to iconify.

Which is stupid by the way, see bug 685759.

Bug seen in vte 0.36/0.38/0.39.
Comment 1 Egmont Koblinger 2014-12-11 16:28:04 UTC 
According to http://invisible-island.net/xterm/ctlseqs/ctlseqs.html, these should save/restore the title, as requested in bug 699819.
Comment 2 Egmont Koblinger 2014-12-11 16:35:36 UTC 
Created attachment 292545 [details] [review]
Fix

Quick fix, with an extra bonus :)
Comment 3 Egmont Koblinger 2014-12-11 16:47:22 UTC 
This approach is still fragile against similar future additions of escapes. I think the big loop should be dropped, and there should be at most 1 action per such escape sequence. This is also what xterm does, try e.g.
echo -ne '\e[13;14t'
In xterm it only reports the window location (13) and not the window size (14).
Comment 4 Egmont Koblinger 2014-12-11 17:04:46 UTC 
Created attachment 292550 [details] [review]
Fix v2
Comment 5 Egmont Koblinger 2014-12-11 19:24:24 UTC 
Committed.