sphinx_plot: Fix regression for 3d plots, suppress axes frame

[mkoeppe]

As reported in https://groups.google.com/forum/#!topic/sage-devel/qivRW2D2Fgg

CC: @egourgoulhon

Component: documentation

Author: Matthias Koeppe

Branch/Commit: 013415d

Reviewer: Eric Gourgoulhon

Issue created by migration from https://trac.sagemath.org/ticket/27481

[mkoeppe]

Branch: u/mkoeppe/sphinx_plot__fix_regression_for_3d_plots__suppress_axes_frame

[mkoeppe]

Commit: 013415d

[mkoeppe]

New commits:

013415d

sphinx_plot: For 3d objects plotted via png, turn off axes frame

[egourgoulhon]

comment:3

Thanks for the fix! Is the ticket ready for review?

[embray]

comment:4

I'd say this is pretty important since without it there's a noticeable, if not mostly aesthetic, regression in the documentation.

[egourgoulhon]

comment:5

Replying to @egourgoulhon:

Thanks for the fix! Is the ticket ready for review?

I've checked the ticket branch: it fixes the issue. So I am ready to give a 'positive review', all the more that the release of 8.7.rc0 is approaching... If you agree, please set the ticket to 'needs review'.

[mkoeppe]

Author: Matthias Koeppe

[egourgoulhon]

comment:7

Thanks!

[egourgoulhon]

Reviewer: Eric Gourgoulhon

[embray]

comment:8

Moving all blocker/critical issues from 8.7 to 8.8.

[vbraun]

Changed branch from u/mkoeppe/sphinx_plot__fix_regression_for_3d_plots__suppress_axes_frame to 013415d