Summary: | JJ: Inconsistency in folder search | ||
---|---|---|---|
Product: | [Unmaintained] kmail | Reporter: | Ruchir Brahmbhatt <ruchir.brahmbhatt> |
Component: | new folder tree | Assignee: | Szymon Stefanek <pragma> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | jtamate, vincent.touffi |
Priority: | NOR | ||
Version: | 1.11.1 | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Unspecified | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: | |||
Attachments: | a patch |
Description
Ruchir Brahmbhatt
2009-03-13 12:29:14 UTC
Confirmed in kde 4.2.65 Created attachment 33877 [details]
a patch
This patch applies to kmail/kmmainwidget.cpp
Hi Vincent, thanks for the patch. While it fixes the issue, I'm not sure if that is the best way to do it. I haven't tested this yet, but doesn't it flicker when saving the tree, or slow down? Another approach might be to not save the visible/hidden state of items in FolderView::saveItemStates(), I see no reason why this should be useful. Could you please use reviewboard for the next iteration of the patch? See http://techbase.kde.org/Contribute/Send_Patches#Reviewboard SVN commit 971322 by tmcguire: Don't hide some folders when restarting KMail after it was quit with having a quick search active. Patch by Vincent Dupont <vincent.touffi at gmail dot com>, thank you! BUG: 187041 http://reviewboard.kde.org/r/743/ M +0 -2 folderview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=971322 SVN commit 972824 by tmcguire: Backport r971322 by tmcguire from trunk to the 4.2 branch: Don't hide some folders when restarting KMail after it was quit with having a quick search active. Patch by Vincent Dupont <vincent.touffi at gmail dot com>, thank you! CCBUG: 187041 http://reviewboard.kde.org/r/743/ M +0 -2 folderview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=972824 |