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

List:       kde-artists
Subject:    [kde-artists] [PATCH]Re: [RFC][Fwd: Re: [RFC][Fwd: Re: Missing
From:       James Richard Tyrer <tyrerj () acm ! org>
Date:       2008-06-19 17:02:40
Message-ID: 485A9130.2000403 () acm ! org
[Download RAW message or body]

James Richard Tyrer wrote:

> And here is the patch [attached].
> 
And here is the RTL patch [attached]] which I have *not* tested.

Could someone that has a RTL language on their system, please help by 
testing this (along with the other patch).

Single patch for both is posted here:

http://home.earthlink.net/~tyrerj/kde/KDE-4/statdard_actions0.diff

Along with a SVG only set of icons to test the patches:

http://home.earthlink.net/~tyrerj/kde/KDE-4/go-icons_KDE4.tar.bz2

Thanks.

-- 
JRT


["statdard_actions-rtl.diff" (text/plain)]

Index: kstandardaction.cpp
===================================================================
--- kstandardaction.cpp	(revision 822252)
+++ kstandardaction.cpp	(working copy)
@@ -108,16 +108,16 @@ KAction *create(StandardAction id, const
 
     if ( QApplication::isRightToLeft() ) {
       if ( id == Prior )
-        iconName = "go-next";
+        iconName = "go-next-page";
 
       if ( id == Next )
-        iconName = "go-previous";
+        iconName = "go-previous-page";
 
       if ( id == DocumentBack )
-        iconName = "go-next";
+        iconName = "go-next-document";
 
       if ( id == DocumentForward )
-        iconName = "go-previous";
+        iconName = "go-previous-document";
     }
 
     QIcon icon = iconName.isEmpty() ? KIcon() : KIcon(iconName);


______________________________________________________________________________
kde-artists@kde.org |  https://mail.kde.org/mailman/listinfo/kde-artists


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