https://bugs.kde.org/show_bug.cgi?id=372964

--- Comment #10 from Alex Bikadorov <alex.bikado...@kdemail.net> ---
Well, if there are icons missing in Oxygen (or any other icon theme) which are
needed by Krusader, it is more a general problem.

I have thought about solving this and Bug 372966:

Two assumptions I take for granted:
* there should never be missing icons for any actions/buttons in the UI. It
simply should not occur in any modern application.
* mixing different icon sets in one application is just ugly and should also
never happen. I can't even remember I have ever seen this anywhere.

And I am really against taking files from another project (Breeze) and copying
them into our project. This is an ugly design decision and leads to more work:
figuring out which icons we need, which icons are obsolete, which icons changed
and need to be updated.

What's left is this (and only this) solution: We make Breeze an official
dependency of Krusader. Because it is the only icon set which contains all
needed icons. And we force using Breeze as icon set if it is installed in the
code; regardless of what the user has set as icon set.
And optionally: Show a warning message on Krusader startup if Breeze is not
installed. Because in this case icons will be missing and the user maybe does
not know why.

Opinions?

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to