Bug 59289

Summary: titlebar does not reflect current mailbox
Product: [Applications] kmail Reporter: Jan De Luyck <bugs+kde>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Debian testing   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Example

Description Jan De Luyck 2003-06-03 18:25:39 UTC
Version:            (using KDE KDE 3.1.2)
Installed from:    Debian testing/unstable Packages
OS:          Linux

When changing to a different mailbox in KMail, the titlebar isn't updated to reflect that change: it stays at the mailbox in which it was when KMail was started.

In the screenshot, the current mailbox is 'Outbox' but the titlebar still says 'jandeluyck @ gmx . net'
Comment 1 Jan De Luyck 2003-06-03 18:26:37 UTC
Created attachment 1709 [details]
Example
Comment 2 Carsten Burghardt 2003-06-03 22:08:38 UTC
Correct, but I only get this behaviour when I change the folder with Next/Prev Unread 
Folder and not when I click on the folder. Nevertheless I?ll have a look at it. 
Comment 3 Till Adam 2003-06-04 21:25:37 UTC
Subject: kdepim/kmail

CVS commit by tilladam: 

Display the correct folder name in the window title bar when changing 
folders via nextUnread(). Carsten, correct fix? 

CCMAIL: 59289-done@bugs.kde.org


  M +1 -0      kmmainwidget.cpp   1.55


--- kdepim/kmail/kmmainwidget.cpp  #1.54:1.55
@@ -1444,4 +1444,5 @@ void KMMainWidget::folderSelectedUnread(
     mHeaders->blockSignals( false );
     mHeaders->highlightMessage( mHeaders->currentItem() );
+    slotChangeCaption(mFolderTree->currentItem());
 }
 


Comment 4 Carsten Burghardt 2003-06-05 19:34:28 UTC
Hmm, should work but it?s more a workaround. The question is: why isn?t the slot called / 
the signal executed? 
I propose to  
- connect the slot to executed (from klistview) instead (currentChanged is not really 
correct as the caption shouldn?t change when you change the keyboard focus but not the 
selection) 
- make sure that the signal is really executed in every situation where the caption should 
change 
Comment 5 Till Adam 2003-06-05 19:45:56 UTC
Because we are in a codepath where signals are blocked. I don't know why, but did 
not want to touch that. If the reason to block signals here is no longer valid, that 
codepath should be cleaned up.