Page MenuHome

Add missing axes to KP* align-view-to-selected
Closed, ArchivedPublicPATCH

Description

This adds meta- and shift-KP* to align the view with the other two orthogonals, and C-KP*, C-M-KP* and C-S-KP* to 180 them (like C-KP1/3/7 does). It also accepts S-M-KP* as an ugly alias for C-M-KP* because the latter is a difficult-to-override hotkey in X.

built and tested against r20092.

Event Timeline

Now I know why KP* didn't do the other axes: view3d_align_axis_to_vector() was broken. Now it's fixed. The new patch version is still incomplete - it doesn't add the new alignments for object-mode KP*, but at least the new version graduates from 'broken' to 'incomplete'.

Version 0003 works: KP* and the S-V menu align the view to any orthogonal axis.

Jim, can you separate your bug-fix from the additional feature?
Thanks

Campbell - yes, the bugfix part is the view.c hunk at the end. The test for it is to try the reverse-view options (C-KP*, and S-V C-1/2/3) on any basis vector.

I regard having the unmodified side view be the only one that views from the -ve direction as a bug. If you want to preserve the original inverted-+ve x-axis view I'll post a new patch with just that; otherwise just `d/view.c/-1`.

Shoulda just done the d/, so here it is in 0001-Fix-PADASTER...(etc)

Hi, trying to update the status of the patch tracker. Please respond if this patch is still viable/useful and needs review or if it can be closed.

Bastien Montagne (mont29) changed the task status from Unknown Status to Unknown Status.Aug 3 2014, 7:09 PM
Bastien Montagne (mont29) claimed this task.

Sounds time to archive this one.