Page MenuHome
Differential D4116

Fix T59633: Impossible to show only x/y axis in ortho front/side view
AbandonedPublic
Actions

Authored by Sebastian Parborg (zeddb) on Dec 21 2018, 3:59 PM.

Details

Reviewers
Brecht Van Lommel (brecht)
Clément Foucault (fclem)
Summary

It seems like the axis (and grid) restriction has been lifted in 2.8. You can now toggle those on and off in ortho view.

However for some reason the z axis didn't get this restriction lifted, so you could toggle the X/Y axis on and off in ortho view, but not the Z axis.

I also noticed that the grid_lines (or gridlines) variable doesn't seem to be used anymore (as the grid is infinite in the 3d view now).

However, I didn't remove it as I'm not sure if this will break older blend files or not. Can I simply remove it from source/blender/makesdna/DNA_view3d_types.h and the rest of the remaining places in the code?

Diff Detail

Event Timeline

Sebastian Parborg (zeddb) created this revision.Dec 21 2018, 3:59 PM
Sebastian Parborg (zeddb) mentioned this in T59633: Impossible to show only x/y axis in ortho front/side view.
Brecht Van Lommel (brecht) added a reviewer: Clément Foucault (fclem).Dec 21 2018, 4:46 PM
Brecht Van Lommel (brecht) added a subscriber: Clément Foucault (fclem).

I don't know if the new behavior is good though. It seems quite useful to be able to disable the axes in perspective view, but then still see them in ortho side views.

@Clément Foucault (fclem), was this change compared to 2.7 intentional?

Sebastian Parborg (zeddb) added a comment.Dec 21 2018, 4:59 PM

Maybe we could add a tickbox to apply the changes to ortho view too then?

Brecht Van Lommel (brecht) added a comment.Dec 21 2018, 5:12 PM

I don't think we need more settings, the automatic behavior worked well enough as far as I know.

Clément Foucault (fclem) added a comment.Dec 21 2018, 6:10 PM

I don't remember this change to be intentional. So we could revert to whatever the old behavior was.

Sebastian Parborg (zeddb) added a subscriber: Campbell Barton (campbellbarton).EditedDec 21 2018, 6:43 PM

It seems like it was @Campbell Barton (campbellbarton) that made this change https://developer.blender.org/rBff39cbdc86ce671c91d0bacf5466ff089d4d5b4f . Don't know if it was intentional or not though...

I'm all for making the settings consistent between view modes. But I can also see the usefulness of the settings not being the same from persp/ortho mode.

Rombout Versluijs (rombout) added a subscriber: Rombout Versluijs (rombout).Dec 24 2018, 4:06 AM

Here's a screengrab to show the issue

Campbell Barton (campbellbarton) abandoned this revision.Jan 2 2019, 8:37 AM

re:

It seems like the axis (and grid) restriction has been lifted in 2.8. You can now toggle those on and off in ortho view.

Reverted to previous behavior since there is no record of this change being intentional.

Thomas Androxman (ThomasAn) added a subscriber: Thomas Androxman (ThomasAn).Feb 9 2019, 8:21 PM

No issues here with the consensus decision other than the UI aspect.

If the grid is not meant to be turned off in ortho views, maybe the option checkbox (under Overlays menu) should become grayed out while in Ortho .... OR.... somehow communicate to the user that this checkbox only affects perspective views (maybe state it under the tooltip).

Revision Contents
Changeset List

PathSize
source/
blender/
draw/
modes/
4 lines
makesrna/
intern/
8 lines

Diff 13015

View Options

source/blender/draw/modes/object_mode.c

Loading...
View Options

source/blender/makesrna/intern/rna_space.c

Loading...