[Kde-imaging] Re: [kde-artists] [PATCH] GwenView: Proper icon for Go to Previous Image & Go to Next Image

[David Miller]

On Tuesday 30 December 2008 08:53:47 James Richard Tyrer wrote:
> The Oxygen team committed new icons: "go-previous-page" & "go-next-page"
> and used those names for the two KDE Standard Actions which GwenView was
> using.  Apparently they didn't understand that these two KDE Standard
> actions were serving two functions.  Actually, I think it is better that
> they do not serve two purposes and that will mean changes to the code
> that will require approval of KDE-Core.
>
> I have committed more generic icons "go-*-view" that are not a page
> specific metaphor which should work for other uses including GwenView.

If generic icons are needed, then what is wrong with the current go-*, or even 
the green arrows used in most other apps?

>
> This small patch should fix the problem:
>
[snip]
>
> and then you will have icons as shown in the attached screen shot and
> will not have to borrow icons intended for other use.
>
> If this is OK, I can commit it for you.