I change all QAction* to KAction* in rkward.h, and the shortcut setting worked.<br>And I find another bug: in R console, backspace key don't work when the cursor is at begin of current line and some part of the line is selected.<br>
Attachment is a patch for these 2 bugs.<br><br><div class="gmail_quote">2009/1/23 Prasenjit Kapat <span dir="ltr"><<a href="mailto:kapatp@gmail.com">kapatp@gmail.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi All,<br>
<div class="Ih2E3d"><br>
On Thu, Jan 22, 2009 at 7:42 PM, Stefan Rödiger <<a href="mailto:stefan_roediger@gmx.de">stefan_roediger@gmx.de</a>> wrote:<br>
</div><div><div></div><div class="Wj3C7c">> Am Tuesday 13 January 2009 08:29:27 schrieb Prasenjit Kapat:<br>
>> Hi All,<br>
>><br>
>> Since KDE 4.2 is about to release, I am test driving it on a separate<br>
>> partition... So, I checked out the trunk of rkward from svn and am<br>
>> giving it a spin... The compile seems to work without any errors (lots<br>
>> of warnings, of course).<br>
>><br>
>> When a script editor window is open, pressing Ctrl+S to save the file,<br>
>> results in a conflict with the "Save Workspace" shortcut. This is<br>
>> observed using the svn trunk of rkward and KDE 4.1.85 on Kubuntu 8.10.<br>
>><br>
>> I presume Thomas is busy so any clarification will take time..<br>
>><br>
>> I was fiddling with the svn code myself. Lines 349 - 352 of rkward.cpp<br>
>> specifically sets Qt:ControlModifier + Qt::ShiftModifier + Qt::Key_S<br>
>> as the action shortcut for "Save Workspace" but, the Shift modifier<br>
>> seems to be ignored, as shown below:<br>
>><br>
>> fileSaveWorkspace = actionCollection ()->addAction<br>
>> KStandardAction::Save, "file_savex", this,<br>
>> LOT(slotFileSaveWorkspace()));<br>
>> fileSaveWorkspace->setText (i18n ("Save Workspace"));<br>
>> fileSaveWorkspace->setShortcut (Qt::ControlModifier +<br>
>> t::ShiftModifier + Qt::Key_S);<br>
>> fileSaveWorkspace->setStatusTip (i18n ("Saves the actual document"));<br>
>><br>
>> One solution (that works) is to change the Save shortcut of "Kate<br>
>> Part"  from Kate to something other than Ctrl+S. But that an<br>
>> outside-of-rkward solution.<br>
>><br>
>> I've filed a bug report on this for rkward, so that it doesn't get<br>
>> lost. Any one seen this issue? Any clue? As of now, I've commented the<br>
>> above four lines in rkward.cpp for my local compilation.<br>
>><br>
>> I can ask on kde devel list about this, but I am not at all familiar<br>
>> with the structure of rkward codes, so it may seem pretty lame!<br>
><br>
> Hi Prasenjit,<br>
><br>
> As you may have seen in the list Roy Qu provided a patch (in trunk now) and according to Thomas it<br>
> might solve your problem. Can you confirm this?<br>
<br>
</div></div>Well, Yes and No!<br>
<br>
No:<br>
Roy's patch was only for the "rkconsole.cpp". So, only the RConsole<br>
part was affected (this is my understanding, which given my C++<br>
knowledge, might very well be wrong).<br>
<br>
Yes:<br>
I did not know this: you could right click on "Save Workspace  Ctrl+S"<br>
menu option to change its shortcut (to, say, Ctrl+Shift+S -- "main"<br>
not "alternate"). This creates the file<br>
~/.kde/share/apps/rkward/rkwardui.rc  wherein this shortcut info is<br>
stored. Similarly, for "Open Workspace  Ctrl+O"  to Ctrl+Shift+O. And<br>
now, all the four shortcuts work as intended.<br>
<br>
So, as you see, the "Yes" part was not influenced by Roy's patch. I<br>
actually tried it - removed Roy's patch and changed the shortcuts.<br>
Works.<br>
<br>
The issue then is, why are the shortcuts from the source code<br>
(rkward.cpp) not honored?  I should add: my previous bug-mail was<br>
under 4.1.85 (beta2) and today I did all this on 4.1.96 (rc1) - so I<br>
guess the KDE APIs are still buggy!?<br>
<br>
One more thing (ala Steve Jobs): If you use Ctrl+S or Ctrl+O before<br>
changing the shortcuts you get the "Ambiguous Shortcut" message. Then<br>
when you try to change the shortcuts, RkWard quits with Fatal Error.<br>
So, you start the application again, and the change the shortcuts<br>
first.<br>
<br>
> Regards<br>
> Stefan<br>
<br>
Thanks to Roy for the RkConsole patch...<br>
--<br>
<font color="#888888">Prasenjit<br>
</font><div><div></div><div class="Wj3C7c"><br>
------------------------------------------------------------------------------<br>
This SF.net email is sponsored by:<br>
SourcForge Community<br>
SourceForge wants to tell your story.<br>
<a href="http://p.sf.net/sfu/sf-spreadtheword" target="_blank">http://p.sf.net/sfu/sf-spreadtheword</a><br>
_______________________________________________<br>
RKWard-devel mailing list<br>
<a href="mailto:RKWard-devel@lists.sourceforge.net">RKWard-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/rkward-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/rkward-devel</a><br>
</div></div></blockquote></div><br>