SolarLune / masterplan

MasterPlan is a project management software / visual idea board software. It attempts to be easy to use, lightweight, and fun.
https://solarlune.itch.io/masterplan
508 stars 46 forks source link

Fix Shortcuts for mac #13

Closed ThinkDigitalSoftware closed 4 years ago

ThinkDigitalSoftware commented 4 years ago

On mac, the "new task" shortcut is Cmd + n, but it's currently set to Ctrl + n

SolarLune commented 4 years ago

Yo! Can you be more specific? Are you talking about the shortcut for sliding Tasks around?

ThinkDigitalSoftware commented 4 years ago

The shortcut for creating a new task . Oh, let me edit the description.

SolarLune commented 4 years ago

I'm not sure I get what you mean - Ctrl+N is the new task shortcut and behaves correctly.

ThinkDigitalSoftware commented 4 years ago

On mac, you usually press command + n for new. But in master plan, it's control +N for new

On Feb 25, 2020 11:54 PM, SolarLune notifications@github.com wrote:

I'm not sure I get what you mean - Ctrl+N is the new task shortcut and behaves correctly.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/SolarLune/masterplan/issues/13?email_source=notifications&email_token=AFPYO7LB3AXFCV42HOLDXG3REYN3NA5CNFSM4KZJFMF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEM7FSNY#issuecomment-591288631, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFPYO7PWMSXCZUWYDQVLYH3REYN3NANCNFSM4KZJFMFQ.

SolarLune commented 4 years ago

OK, I wasn't sure if that's what you meant or not. I'll have to think about how best to approach this one, as that would mean a different key for Mac as a platform specifically for any shortcut that uses Control.

ThinkDigitalSoftware commented 4 years ago

Is that complicated to implement?

On Feb 26, 2020 12:40 AM, SolarLune notifications@github.com wrote:

OK, I wasn't sure if that's what you meant or not. I'll have to think about how best to approach this one, as that would mean a different key for Mac as a platform specifically for any shortcut that uses Control.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/SolarLune/masterplan/issues/13?email_source=notifications&email_token=AFPYO7MAWDCO623QTURR76LREYTIVA5CNFSM4KZJFMF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEM7JXZQ#issuecomment-591305702, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFPYO7KPE5NM2CCBDVECS5LREYTIVANCNFSM4KZJFMFQ.

SolarLune commented 4 years ago

Not really, but it isn't necessarily trivial, as I need to change all shortcuts that use Ctrl to use the Super key when running on Mac specifically. That's an additional layer of redirection that I'll have to add to the existing shortcut system.

ThinkDigitalSoftware commented 4 years ago

Ah ok

On Feb 26, 2020 8:59 AM, SolarLune notifications@github.com wrote:

Not really, but it isn't necessarily trivial, as I need to change all shortcuts that use Ctrl to use the Super key when running on Mac specifically. That's an additional layer of redirection that I'll have to add to the existing shortcut system.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/SolarLune/masterplan/issues/13?email_source=notifications&email_token=AFPYO7L4P3SBP22K6P2RS5TRE2NWLA5CNFSM4KZJFMF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENBBBXQ#issuecomment-591532254, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFPYO7MTBJY6MWMXOIM64OLRE2NWLANCNFSM4KZJFMFQ.

SolarLune commented 4 years ago

So this should be resolved as of the linked commit; I added the ability to use both Control as well as the Super keys for shortcuts.

ThinkDigitalSoftware commented 4 years ago

Awesome! Thanks

On Mar 5, 2020 7:59 PM, SolarLune notifications@github.com wrote:

So this should be resolved as of the linked commit; I added the ability to use both Control as well as the Super keys for shortcuts.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/SolarLune/masterplan/issues/13?email_source=notifications&email_token=AFPYO7P6VMZJZ6JD2D7YGJDRGBYLNA5CNFSM4KZJFMF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEN77OHA#issuecomment-595588892, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFPYO7KOIZ2KE37BOXE7J3DRGBYLNANCNFSM4KZJFMFQ.