[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: patch: konsole
From:       Lotzi Boloni <boloni () pacbell ! net>
Date:       2001-06-24 15:55:52
[Download RAW message or body]

On Sunday 24 June 2001 06:27, Simon Hausmann wrote:
> On Sun, Jun 24, 2001 at 03:13:59PM +0200, Thomas Zander wrote:
> > On Sun, Jun 24, 2001 at 02:56:16PM +0200, Simon Hausmann wrote:
> > > Hi,
> > >
> > > The attached patch correctly saves/restores the user-specified caption
> > > (the one specified through that special xterm escape sequence) when
> > > switching between sessions. Please review.
> >
> > Consider me stupid, but this looks like a new feature to me...
> >
> > Sorry Simon ;-)
>
> I consider it a bugfix because it fixes the current existing support for
> user specified captions when used together with multiple sessions ;-)
>
> For example if you have a prompt that has escape sequences in it to set
> the konsole title to the current working directory and you're working
> with multiple sessions then with the current konsole the title is not
> saved/restored correctly across multiple sessions. I consider that to be
> a bug because it's confusing that I can't rely on the window caption to
> detect which shell is under the window, although it is supposed to
> work correctly (as I added those escape sequences to the prompt) .
> (and the whole point why I put those into my prompt is in order to
> _rely_ on them when the sessions are listed in the taskbar)

   Hi guys 

   This IS a bugfix, the original title field was intended to work this way.
I think that it was a somewhat bad decision when the "Title" of the session 
(which appears in the button), was messed up with the "Caption" of the 
session. 

   BTW, we fixed exactly this thing with a collegue of mine, Bill Bouma 
several days ago, but the drawback of that fix seemed to be that at session 
restore, the full header, instead of the session name is restored, so I 
didn't commit it (hoping that I might have some time to make a cleanup around 
there). If Simon's patch is working it would be nice to put it in, as this is 
certainly a bug. 

     cheers      
              Lotzi 

[prev in list] [next in list] [prev in thread] [next in thread]