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

List:       kde-multimedia
Subject:    Bug#36303: Gui stuff.
From:       Neil Stevens <neil () qualityassistant ! com>
Date:       2001-12-19 21:06:31
[Download RAW message or body]

On Wednesday December 19, 2001 12:08, Thomas Zander wrote:
> - in the effects dialog the names used for the effect have no meaning to
> most (99%) of the users; a text explaining what they are would be a huge
> help.

Have arts add such a line to the mcopclass files, and I'll be glad to 
re-add the part of the UI that displayed the descriptions.  Right now the 
descriptions simply don't exist anywhere to display.

> - in the effects dialog the 'remove effect' icon is a close icon.
> It should be the remove icon (the red cross on a piece of paper)

Can you commit this yourself, since you seem to know which icon is 
preferred.  Thanks

> - the window-close icon of (at least) the excellent skin

skin? *cough, cough*

> closes the application as pointed out before this is not correct and not
> according to the styleguide, yet you choose to change it again. ???
> Maybe you don't respect the styleguide ?

See my comments in that thread.  The styleguide, in that section, only 
addresses document-centered apps, and the distinction between close and 
quit made there makes absolutely no sense in the context of noatun.  I 
would suggest that in non-document apps, like noatun or ksirc, Close 
doesn't belong in a controller window, and Quit should quit the app like 
the standard says.

Furthermore, the styleguide certainly does Not say that applications 
should act differently because they have an optional systray icon, that I 
saw.

-- 
Neil Stevens
neil@qualityassistant.com

Don't think of a bug as a problem.  Think of it as a call to action.
_______________________________________________
kde-multimedia mailing list
kde-multimedia@mail.kde.org
http://mail.kde.org/mailman/listinfo/kde-multimedia
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic