Bug 115611

Also, for consistency, the "Delete folder" should be re-added to the RMB menu in the folder tree. Everywhere else the delete action is in the RMB menu, just kmail's folder tree doesn't have it. 

Reinhold

*** Bug 116470 has been marked as a duplicate of this bug. ***

That sounds a lot like the old properties dialog which we splitted because of 
input from the usability team...

It might have made the Properties dialog simpler, but it's made it a lot worse for someone trying to configure the properties on a folder.

As I said, I have now to go to 3 or 4 different dialogs. It gets especially annoying since KMail loses the folder settings after a crash... 3 * 50 folders is a LOT of dialogs to go through.

I agree, the new system is horribly confusing.

I tried to report this on the usability mailing-list:
http://comments.gmane.org/gmane.comp.kde.usability/14947

This is their attitude on the subject
"To be blunt, this is really just noise.  You're not going to get behavior 
reverted that was agreed on by the developers and the usability team by 
bringing it up here.

-Scott"

But it has happened before, like the Delete folder entry in the RMB menu, as mentioned in comment #1

*** Bug 114893 has been marked as a duplicate of this bug. ***

*** Bug 109170 has been marked as a duplicate of this bug. ***

Looks like Openusability closed the Kmail project (what a pity!). Maybe it's time to make a REAL usability study of the app, and make it work again as it should.
Any volounteers? I lift up my hand!

For full dialog I opt to merge them into folder properties (as tab) and to _remove_ them from the folder menu, but for single options (see folder menu) I opt to group them in the properties dialog but also to keep them in this menu --> because it is faster to just switch one single option (like assign shortcut, or load external references) and continue working.

See http://lists.kde.org/?t=123368147700003&r=1&w=2
and http://reviewboard.kde.org/r/48/

This is still a contentious usability issue, so please comment/vote further on this bug if you would like to see any further action...

SVN commit 927964 by marten:

Add a "Maintenance" tab to the folder properties dialogue.  This gives some
information about the folder, its file and index locations and sizes, and the
content and messages.

For mbox folders, the compaction state and a button to do it is here (still
also in the folder context menu).  A folder can be compacted (at the user's
own risk) even if it was previously marked "unsafe".

Other dangerous operations ("Rebuild Index" and "Rebuild IMAP Cache") have
been moved from the context/menubar menus to here.

Moved the "mark_all_as_read" action back to the top of the folder popup menu,
as suggested by aseigo.

Added some hooks (KMFolderDialog::setPage() and KMMainWidget::PropsPage enum)
to embed the other folder properties into this dialogue, if there can ever be
agreement on the usability of this.  So it's not a fix for bug 115611, but it's
a start...

CCBUG:115611


 M  +1 -2      folderdialogacltab.cpp  
 M  +0 -1      folderdialogacltab.h  
 M  +2 -3      folderdialogquotatab.cpp  
 M  +0 -1      folderdialogquotatab.h  
 M  +10 -0     folderstorage.cpp  
 M  +6 -0      folderstorage.h  
 M  +7 -30     folderview.cpp  
 M  +0 -6      folderview.h  
 M  +1 -1      identitydialog.cpp  
 M  +2 -5      kmail_part.rc  
 M  +302 -16   kmfolderdialog.cpp  
 M  +56 -4     kmfolderdialog.h  
 M  +20 -10    kmmainwidget.cpp  
 M  +14 -1     kmmainwidget.h  
 M  +2 -5      kmmainwin.rc  


WebSVN link: http://websvn.kde.org/?view=rev&revision=927964

*** Bug 298894 has been marked as a duplicate of this bug. ***

I would like to see Expiry Properties merged into Folder Properties, as I explained in duplicate bug 298894. Having it separate is confusing, which *reduces* usability.

Thank you for your feature request. Kmail1 is currently unmaintained so we are closing all wishes. Please feel free to reopen a feature request for Kmail2 if it has not already been implemented.
Thank you for your understanding.

Reopening because still relevant to KMail2.
Product and version updated.

*** Bug 305373 has been marked as a duplicate of this bug. ***

Git commit d651859dbd410da40d2dd53d6156fa721839af0b by Jonathan Marten.
Committed on 21/11/2012 at 17:10.
Pushed by marten into branch 'master'.

Embed "Mailing List Management" into "Folder Properties"

The original action (in the "Folder" menu bar menu) remains
present, it opens the "Folder Properties" dialogue at the
appropriate tab.

This requires commit ea9a9a132e09d2ac1bd359bcccf68cdde3b3c566
to kdepimlibs.
REVIEW:107341

M  +1    -1    kmail/CMakeLists.txt
R  +108  -130  kmail/collectionmailinglistpage.cpp [from: kmail/mailinglistpropertiesdialog.cpp - 058% similarity]
A  +91   -0    kmail/collectionmailinglistpage.h     [License: GPL (v2)]
M  +23   -8    kmail/kmmainwidget.cpp
M  +2    -0    kmail/kmmainwidget.h
D  +0    -86   kmail/mailinglistpropertiesdialog.h

http://commits.kde.org/kdepim/d651859dbd410da40d2dd53d6156fa721839af0b

Git commit e1d9e4cde408fdcf7387b9f980c068d4d9e1e724 by Jonathan Marten.
Committed on 18/01/2013 at 15:27.
Pushed by marten into branch 'master'.

Embed "Assign Shortcut" into "Folder Properties"

The original actions (in the "Folder" menu bar menu and folder
list popup) remain present, they open the "Folder Properties"
dialogue at the appropriate tab.
REVIEW:108447

M  +1    -1    kmail/CMakeLists.txt
A  +108  -0    kmail/collectionshortcutpage.cpp     [License: GPL (v2)]
A  +55   -0    kmail/collectionshortcutpage.h     [License: GPL (v2)]
D  +0    -100  kmail/foldershortcutdialog.cpp
D  +0    -77   kmail/foldershortcutdialog.h
M  +4    -7    kmail/kmmainwidget.cpp

http://commits.kde.org/kdepim/e1d9e4cde408fdcf7387b9f980c068d4d9e1e724

Jonathan, can we resolve this one now?

One more dialogue to embed (expiry).  Will do that very soon, then the bug can be closed...

Git commit 8ad83768ca26a6cbded59d5f7714bd0d3e1d0b26 by Jonathan Marten.
Committed on 19/01/2013 at 18:30.
Pushed by marten into branch 'master'.

Embed "Expiry" into "Folder Properties"

The original "Expire" actions (in the "Folder" menu bar menu and
folder list popup) remain present, they open the "Folder Properties"
dialogue at the appropriate tab.

As noted in comments in KMMainWidget::setupActions(), there is
still inconstency between the wording of the action and what it does.

Hooray, a seven year old wish finally implemented!
REVIEW:108485

M  +5    -6    kmail/kmmainwidget.cpp
M  +1    -1    mailcommon/CMakeLists.txt
A  +275  -0    mailcommon/collectionexpirypage.cpp     [License: GPL (v2)]
A  +81   -0    mailcommon/collectionexpirypage.h     [License: GPL (v2)]
D  +0    -244  mailcommon/expirypropertiesdialog.cpp
D  +0    -47   mailcommon/expirypropertiesdialog.h

http://commits.kde.org/kdepim/8ad83768ca26a6cbded59d5f7714bd0d3e1d0b26

> Hooray, a seven year old wish finally implemented!

Congratulations!

Are you considering removing the old menu items in a future release? It might be confusing having duplicate items. I think I'd be confused.

(In reply to comment #24)
> > Hooray, a seven year old wish finally implemented!
> 
> Congratulations!
> 
> Are you considering removing the old menu items in a future release? It
> might be confusing having duplicate items. I think I'd be confused.

No we will not remove them.
Regards

You could remove them per-user or locally, though, by editing the rc file - and we could maybe convert the context menu to an XMLGUI container so that it can be modified in the same way.

Correction - it already is.  So just edit the kmmainwin.rc file.