[Octave-bug-tracker] [bug #39433] subplot sizes wrong

[Ben Abbott]

Follow-up Comment #3, bug #39433 (project octave):

Michael,

My guess is that one of both of the changesets below tiggered the regression.

Initialization and updating of tightinset and looseinset properties.
http://hg.savannah.gnu.org/hgweb/octave/rev/e528beb2d774

Include "looseinset" in axes::properties::update_units ().
http://hg.savannah.gnu.org/hgweb/octave/rev/5893eb92361a

However, these actually fixed bugs, so I expect the problem is somewhere else.
 Iv'e been looking at graphics.cc (axes::properties::sync_positions).  My c++
is rather poor, so the going is slow.  However, my impression is that there is
some code there that is extraneous (looseinset is set twice, the first time
looks to be over-ridden by the second).  Also the syncing is for the axes'
outerposition, while Matlab syncs the axes' position.  This is where the
problem is (I think?).

For compatiblity,

* subplot(n, m, k, "allign"); should treat the initial position property as
fixed and allow the axes labels and ticklabels to encroach upon neighboring
axes (implied activepositionproperty = position). 
* subplot(n, m, k); should treat the initial position property as flexible and
allow the positions to shrink to prevent the axes labels and ticklabels from
encroaching upon neighboring axes (implied activepositionproperty =
outerposition).
* In either case the subplot positions are aligned with the neighbors.

I'm skeptical I'll be able to fix this quickly.  Especially since I'm not
certain that the problem doesn't go beyond sync_positions.  If you're upto
looking into this, I could use the help.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?39433>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/