From: Thorsten O. <ha...@ya...> - 2016-02-05 02:22:46
|
> that whole keyboard handling thing in the desktop needs a rewrite so that > it does a lookup in the menu tree for the matching key If you are going to make the shortcuts configurable, you should rather do it the other way around, changing the menu entries accordingly. Allowing the user to change the RSC and then expect the desktop to make up correct translations to keyboard shortcuts is asking for trouble. I once made the mistake to add code to ORCS to check for duplicate shortcuts, you won't believe what kind of strings are used for some keys in some programs (e.g "Ret." "Return", "Ent", "Enter", and that in all combinations of upper and lowercase) Roger Burrows <an...@xp...> schrieb am 1:37 Freitag, 5.Februar 2016: On 4 Feb 2016 at 14:42, Vincent Rivière wrote: > On 04/02/2016 13:48, Thorsten Otto wrote: > > Using objc_delete() or setting OF_HIDETREE is not enough in this case. > > That will leave a "hole" in the menu. > > You are completely right. I experienced that last night. > > Furthermore, there is some code in EmuDesk which dynamically > enable/disable the items in the File menu. That made me almost crazy. > I think that the easiest way of handling the Shutdown item is to disable it (grey it out) when it's inappropriate, rather than try to remove it. No point in making the code more complex than it needs to be. Don't forget to ignore the ctl-Q in hndl_keybd() as well. BTW, that whole keyboard handling thing in the desktop needs a rewrite so that it does a lookup in the menu tree for the matching key ... but that's another project (along with the ability for the user to assign shortcuts, like TOS3 or TOS4). Roger ------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140 _______________________________________________ Emutos-devel mailing list Emu...@li... https://lists.sourceforge.net/lists/listinfo/emutos-devel |