Direction arrows.
James Richard Tyrer
tyrerj at acm.org
Mon Jun 25 22:39:04 BST 2007
Diego Iastrubni wrote:
> ביום שני, 25 ביוני 2007, נכתב על ידי James Richard Tyrer:
>> I am aware of this issue and these two KDE3 icons: "previous" & "next"
>> shouldn't be interchanged for RTL language/locales. Since these are not
>> the icons for a KDE Standard Action, this is an issue with individual
>> applications. And yes, this should be in the HIG.
>
> Richard,
>
> If you talk about the icons on those buttons (in LTR, next points to the right
> and in RTL ro the left), you missed my point. I am talking about the order of
> those buttons.
OK, I think that I understand now.
Rather than have standard actions: "FindNext" and "FindPrevious" we
should call them: "FindRight" and "FindLeft". The icons should _always_
point Right and Left (respectively) and should not change for RTL.
However, the function, menu string and tooltip should change.
That is for LTR, FindRight should have the function FindNext, but for
RTL FindRight should have the function FindPrevious.
Which means that this code is not needed:
if ( QApplication::isRightToLeft() ) {
if ( id == Prior )
iconName = "find-next";
if ( id == Next )
iconName = "find-previous";
}
but a considerable rewrite of kstandardaction.cpp and the header files
is needed.
And this should be the same for all right/left text related standard
actions. GoBack & GoForward should be replaced by GoLeft & GoRight --
this code is also wrong:
case Back:
sLabel = i18nc( "go back", "&Back");
if ( QApplication::isRightToLeft() )
iconName = "go-next";
break;
case Forward:
sLabel = i18nc( "go forward", "&Forward" );
if ( QApplication::isRightToLeft() )
iconName = "go-previous";
break;
because it changes the icon and it should change the string -- and the
missing ViewPrevious & ViewNext should be called ViewLeft & ViewRight.
--
JRT
_______________________________________________
Kde-usability-devel mailing list
Kde-usability-devel at kde.org
https://mail.kde.org/mailman/listinfo/kde-usability-devel
More information about the kde-core-devel
mailing list