Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace DEL with Backspace for OSX #994

Merged

Conversation

manongjohn
Copy link
Collaborator

This addresses an issue for MacOS users where the Delete key on mac keyboard does nothing because it is internally identified as Backspace and actions are not typically defined against it.

The following changes were made for macOS builds only:

  • Replaced the default Delete keyboard shortcut DEL with Backspace

    • This generally impacts wherever a menu shortcut command or action uses the T2D delete command option.
  • Anywhere DEL is hardcoded for a particular action and there is no Backspace action defined, the logic has been switched to work with Backspace instead

    • Hex Color Name Editor - Removing entries
    • Motion Path Panel - Graph widget - Removing curve points
    • CurvesFx - Removing curve point
  • Anywhere the Backspace is hardcoded for a particular action and there is no DEL action defined, the logic will remain the same

    • Script console - removing text
    • Stop Motion Panel - Numpad shortcuts active option switches Backspace shortcut to Stop Motion Remove Frame command
  • Anywhere a DEL action and a Backspace action is hardcoded with different actions, the logic will remain the same

    • Type Tool - text edit box

@manongjohn manongjohn added the 1.4 Included in Version 1.4 label May 9, 2022
@artisteacher
Copy link
Contributor

This works beautifully! This makes life so much easier on macOS!

@manongjohn manongjohn merged commit d8503e5 into tahoma2d:master Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.4 Included in Version 1.4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants