<div class="gmail_quote">On Fri, May 11, 2012 at 8:42 PM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

On Friday 11 May 2012 19:57:11 Mark wrote:<br>
> >  KShortcutManager::setDefaultShortcut(newAct, Qt::Key_F6);<br>
><br>
> I like this idea!<br>
<br>
Good, me too :-)<br>
<div class="im"><br>
> I was actually intending to use the Shortcut Editor but i<br>
> didn't look into specifics yet.<br>
><br>
> With dynamic property you're talking about<br>
> <a href="http://qt-project.org/doc/qt-4.8/qobject.html#setProperty" target="_blank">http://qt-project.org/doc/qt-4.8/qobject.html#setProperty</a> right?<br>
<br>
</div>Yes. Just like the current KAction code already does, to store the default<br>
primary and alternate shortcuts.<br></blockquote><div><br></div><div>Ah, oke. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> > krazy would basically flag any direct call to QAction::setShortcut as a<br>
> > "bad<br>
> > hardcoded shortcut, you should use KShortcutManager instead".<br>
><br>
> What you're suggestion is that _all_ KAction obects go through<br>
> KShortcutManager.<br>
<br>
</div>All those with a default shortcut set by the application, yes, obviously.<br>
<div class="im"><br>
> Might be good for an ideal world situation but i don't<br>
> really know if this is desirable. I just don't know for this part.<br>
<br>
</div>Why not? How else would this work? I'm not sure what the reservation is, here.<br>
<br>
The alternative is hardcoding shortcuts that the user cannot modify, I don't<br>
see why we would want to do that.<br></blockquote><div><br></div><div>The only reservation i have is the case where some developer comes by and makes a KDE application with actions that aren't defined yet in Shortcut Manager. Then that developer might just wnat to use setShortcut which is fine right? Yet krazy is going to blame the developer that he shouldn't do that. That's my only reservation. If that is a non issue then i have no reservation ^_^</div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> I do want to get back on KInputShortcut and KInputSequence about them being<br>
> not KDE specific and should end up in Qt. I fully agree and there has been<br>
> discussion about that some time ago on the Qt mailing lists. I actually<br>
> started that discussion so i know what i'm talking about :)<br>
<br>
</div>Ah, great. Sorry for my bad memory ;) </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> The idea back<br>
> then was that global shortcuts should use the platform plugin stuff and<br>
> should use some kind of global shortcut storage mechanism. The thing that's<br>
> called "Shortcut Editor" in KDE. Mac and Windows sadly don't have such<br>
> applications. So the Qt implementation would be quite... difficult. KDE<br>
> already has some more infrastructure with global shortcuts so there it's<br>
> "fairly easy" to get the suggested approach working.<br>
<br>
</div>I'm not talking about global shortcuts, but about input shortcuts (i.e.<br>
including gestures and mouse+keyboard combinations in there).<br>
Isn't that the idea of KInputShortcut?<br></blockquote><div><br></div><div>Oke, i'm over eager again ;)</div><div>Lets - for the sake of simplicity - ignore global shortcuts for now. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
I'm leaving global shortcuts completely on the side, they are a whole other<br>
can of worms. One step at a time :-)<br></blockquote><div><br></div><div>Deal </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
--<br>
David Faure, <a href="mailto:faure@kde.org">faure@kde.org</a>, <a href="http://www.davidfaure.fr" target="_blank">http://www.davidfaure.fr</a><br>
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5<br>
<br>
</div></div></blockquote></div><br><div>Another question since we seem to be on the "we both agree" track here. How about the current KShortcut and KAction stuff along with deprecation. What should be deprecated, Where should it be deprecated (in 4.9 is my preference) and how about my current patch in reviewboard? Is that one becoming obsolete with this idea? (seems like it)</div>

<div><br></div><div>Mark</div>