Bug 136396 - move dialog: strange gray background for some folders
Summary: move dialog: strange gray background for some folders
Status: RESOLVED FIXED
Alias: None
Product: kmail
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
: 148887 150845 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-10-27 22:06 UTC by Maciej Pilichowski
Modified: 2008-01-27 01:29 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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. ***