Page MenuHome

Fix T88068: Changed default Alt-I behavior to be more intuitive
ClosedPublic

Authored by Xing Liu (xing) on Jun 8 2021, 3:59 AM.

Details

Summary

This remaps Alt-I from delete_key_v3d_exec to delete_key_exec. On reproducing T88068, this fixes both the "location & rotation" keying set and the "power" keying set.

Diff Detail

Repository
rB Blender
Branch
arcpatch-D11528 (branched from master)
Build Status
Buildable 15127
Build 15127: arc lint + arc unit

Event Timeline

Xing Liu (xing) requested review of this revision.Jun 8 2021, 3:59 AM
Xing Liu (xing) created this revision.
Xing Liu (xing) retitled this revision from Fix T88068: Changed default Alt-I behavior to be more intuitive --- This remaps Alt-I from delete_key_v3d_exec to delete_key_exec. On reproducing the report, this fixes the first bug (location and rotation keying set) but does not fix the power... to Fix T88068: Changed default Alt-I behavior to be more intuitive.Jun 8 2021, 4:00 AM
Xing Liu (xing) edited the summary of this revision. (Show Details)

Undo accidentally deleted line

Fix line misalignment

Xing Liu (xing) edited the summary of this revision. (Show Details)Jun 8 2021, 11:26 PM

This should probably also be addressed in the Industry Standard keymap then? That's stored in industry_compatible_data.py.

I'll apply that change when committing this patch. Thanks for the contribution!

This revision is now accepted and ready to land.Jun 11 2021, 11:05 AM
Sybren A. Stüvel (sybren) edited the summary of this revision. (Show Details)
  • Apply the same change to the Industry Compatible keymap
This revision was automatically updated to reflect the committed changes.