#1847: Preliminary qt->trunk merge patch
-----------------------+----------------------------------------------------
Reporter: Buginator | Owner:
Type: patch | Status: new
Priority: critical | Milestone: unspecified
Component: other | Version: unspecified
Keywords: | Operating_system: All/Non-Specific
Blockedby: | Blocking:
-----------------------+----------------------------------------------------
Comment(by dak180):
Replying to [comment:3 Safety0ff]:
> To dak180 and Buginator regarding Mac keymaps:[[BR]]
> Have you guys tried switching all the reference to Ctrl and meta for
mac?
> The wz code seems to expects KEY_LMETA to be the command key, so I think
this would keep things consistent.[[BR]]
>
>[[BR]]
>...[[BR]]
>[[BR]]
>
> Also, I wouldn't call this a "HACK" (like you do in the comment,)
because it is covered in the Qt documentation.
I would not only call it a hack but a hack that is both ineffective and
wrong.
All the shortcuts that use ctrl elsewhere should be using cmd on the mac;
it is what is expected on the platform.
As a excellent example of this: the standard key combo for paste on the
mac is cmd+v, not ctrl+v.
Used properly (at least according to the docs) Qt handles all of this on
its own, and in fact it was doing so, the only issue being that the keymap
screen does not display properly (which these changes do not fix), the
actual inputs worked correctly; that the display is not working says to me
that we are most likely using something incorrectly.
I would guess that it is related to the thing that makes it so (at least
on the mac) no key combos using alt work.
--
Ticket URL: <http://developer.wz2100.net/ticket/1847#comment:5>
Warzone 2100 Trac <http://developer.wz2100.net/>
The Warzone 2100 Project
_______________________________________________
Warzone-dev mailing list
[email protected]
https://mail.gna.org/listinfo/warzone-dev
