[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-usability
Subject:    Re: The new KFileDialog
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2005-02-23 19:05:53
Message-ID: 200502231205.55847.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Wednesday 23 February 2005 02:43, Thomas Zander wrote:
> See attached screenshot of what I am seeing now; with my old color scheme.

either you haven't updated your CVS recently to catch the fix i made where it 
was unintentionally using a lightened version of  the highlight color, or 
your colour scheme is really, really bad. it just uses your highlight colour 
for the background, and the highlighted text colour for the text. in other 
words: your colourscheme colours.

> The problem I see is
> a) the sidebar is not distinctive from the rest of the dialog anymore.

it's set off by a subtle border; and it's got icons with text. i don't know 
why we need it more obvious than that. do people using it suddenly not 
understand that the icon labelled "Home" doesn't take them to their Home? or 
are we moaning here because it's different and so inventing possible, 
theoretical problems that may exist, possibly?

> b) we now have the same text-selected color for the normal text
> (mostly black) as well as for the text that is normally positioned on
> top of a really dark color.
> In effect having 3 light colors trying to be contrasting..

as i noted above, either your CVS is too old or your colour scheme is screwed.

> I suggest;
> 1) you use the normal 'selected text' color for the selected title

http://webcvs.kde.org/kdelibs/kio/kfile/kurlbar.cpp?view=markup

            if ( isCurrent() || isSelected() ) {
                p->setPen( box->colorGroup().highlight().dark(115) );
                p->drawText( x + ( QApplication::reverseLayout() ? -1 : 1),
                             y + 1, visibleText );
                p->setPen( box->colorGroup().highlightedText() );
            }

            p->drawText( x, y, visibleText );

as you can see, that's exactly what it does in CVS HEAD.

> 2) you use a color for the background of the sidebar thats more
> towards the selected text color.  So take the background and take the
> selected text color and use a color thats 30% between them.

and have yet another colour cue that appears nowhere else in KDE. i really 
don't understand why we want some totally different colour cue here. it's 
unecessary. we have this ONE widget that previously has been painted 
completely differently from everything else (mostly to look like win32, 
AFAICT) and which took focus from the central area.

did you know that the preview area ALSO uses the window background. maybe we 
should change that too, and have globs of light and dark colours everywhere. 
=(

-- 
Aaron J. Seigo
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

[Attachment #5 (application/pgp-signature)]

_______________________________________________
kde-usability mailing list
kde-usability@kde.org
https://mail.kde.org/mailman/listinfo/kde-usability


[prev in list] [next in list] [prev in thread] [next in thread]