Bug 136396

Summary: move dialog: strange gray background for some folders
Product: [Applications] kmail Reporter: Maciej Pilichowski <bluedzins>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ch.trassl, demerson3x, tero
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: openSUSE   
OS: Linux   
Latest Commit: Version Fixed In:

Description Maciej Pilichowski 2006-10-27 22:06:03 UTC
Version:            (using KDE KDE 3.5.5)
Installed from:    SuSE RPMs

It looks like the dialog is dirty ;-) I know it sounds funny, but the visual effect is annoying. If I remember correctly 3.5.4 or 3.5.5 introduced this bug (minor, I know).
Comment 1 Allen Winter 2006-10-27 22:17:30 UTC
Maciej,

You need to join the KDE usability team :>

I never noticed before, but you sure are correct.  There is a very light gray background on some folders and not others.
Comment 2 Ingo Klöcker 2006-10-27 23:48:44 UTC
Could that be the alternative background color? As in the message list, but with a wrong alternative background color?
Comment 3 Maciej Pilichowski 2006-10-28 09:21:50 UTC
Allen, ok, thank you ;-) I will check the general KDE-UT mailing list. I would like to help, so if the traffic doesn't kill me...
Comment 4 Bram Schoenmakers 2006-10-28 10:20:07 UTC
The grey color background does not match the alternative list color in the KDE wide scheme. I fail to see how this is an usability issue, though.
Comment 5 David L Emerson 2007-11-01 11:07:01 UTC
This is a usability issue if you've got an inverted color scheme, and every other item is displayed as white text on a white backgroud ;)
Comment 6 Maciej Pilichowski 2007-11-01 11:18:37 UTC
But I don't have inverted color scheme. 

Btw. I use Kmail from OS 10.3 and there is no regular KMail available, only enterprise edition and it seems to be fixed here.
Comment 7 Thomas McGuire 2007-11-01 14:07:31 UTC
What dialog are you actually referring to? The move dialog of filters (Settings->Configure Filters->Filter Actions->Move Into Folder) ?
Comment 8 Maciej Pilichowski 2007-11-01 14:18:52 UTC
Thomas, click LMB for example in inbox on any message and then use "move message to folder". The dialog will appear -- so this is that dialog (maybe it is the same dialog you pointed out).
Comment 9 Thomas McGuire 2007-11-01 14:26:12 UTC
Ah yes, that is the same dialog like the one I mentioned above and also the same as the "Jump to folder" dialog.
The code uses custom alternate row coloring, which is really buggy.
I've already removed that code for trunk (revision 656303).
Right now, that dialog has no alternate row coloring at all, but will get it again as soon as it is completely ported to Qt4 (which has built-in support for it),

Because trunk does not have this bug, I'll mark it as fixed.
Comment 10 David L Emerson 2007-11-01 17:52:37 UTC
This may be flogging the issue to death, but here's a list of the places I see this behavior...

"Find messages" > "Search only in..." > (Click folder icon) > "Select Folder" dialog

Settings > Configure kmail > Identities > (choose one) > Modify > Advanced > (choose Sent-mail or Drafts folder) > "Select Folder" dialog

similar for Accounts > Receiving > (choose an imap account) > Modify > Trash Folder

...and of course the move dialog. And perhaps there are other places I missed, but assumably this buggy code is only implemented once and called from several places. Hopefully, anyway!

As for a usability issue, *I* have an inverted color scheme, thus my comment...
http://david9.freepgs.com/i/kmail_alt_color.png

There, now it's thoroughly beaten to death ;D

Thanks for the fix!
Comment 11 Thomas McGuire 2007-11-01 17:58:17 UTC
> This may be flogging the issue to death, but here's a list of the
> places I see this behavior...
> [..] but assumably this buggy code is only implemented once and called
> from several places.

Yes, all the places you listed share the same code.
Comment 12 Thomas McGuire 2007-11-22 17:57:32 UTC
*** Bug 150845 has been marked as a duplicate of this bug. ***
Comment 13 Thomas McGuire 2008-01-27 01:29:21 UTC
*** Bug 148887 has been marked as a duplicate of this bug. ***