rect toolbar width/height numbers do not consider parent transform

[Thomas Holder]

the toolbar for the rect tool shows W/H/Rx/Ry. These numbers do not consider the parent transform but show directly what is saved with its svg attributes height/width/rx/ry.

The selector toolbar shows the correct width/height.

[su_v (suv-lp)]

reproduced with Inkscape 0.47+devel r9391 on OS X 10.5.8

steps to reproduce:
1) draw rectangle
2) group it, scale the group
3) select the rectangle with the rect tool:
   W/H/Rx/Ry show initial, untransformed values
4) ungroup
5) select the rectangle with the rect tool:
   W/H/Rx/Ry show transformed values

related to or duplicate of
Bug #196624 “Rounded corners do not scale when 2 rectangles with R=4 grouped”
<https://bugs.launchpad.net/inkscape/+bug/196624>

[David Mathog (mathog)]

This bug is almost 6 years old!

It would be easy enough to walk back up the stack and find the absolute scale to use. However, with a single scale value the results would not be meaningful if any of the traversed transforms differentially scales X and Y or is skew. In a situation like that the "rectangle" might appear on the screen as an arbitrary convex quadrilateral, in which case width and height mean what? In this scenario perhaps these fields should be grayed out or a "is later transformed" indicator lit?

Also in cases where pure scaling applies but the transform rotates the base rectangle by 90 degrees, which is then Width and which is Height? Presumably it would be on the base rectangle, but the UI might confuse a user considerably when changing height makes the rectangle on the screen wider.

When these problem cases arise, and at the expense of a much more complex control, one could put up a small image with say a blue arrow along the base of the reference rectangle, and a red arrow along the left side, with those arrows also drawn overlaying the final drawing. That would make it evident how changes in the control were mapping onto the figure.

[Alvin Penner (apenner)]

fix committed to rev 13964

[su_v (suv-lp)]

Backport of r13964 for 0.91.x.

[su_v (suv-lp)]

Fix backported to 0.91.x in rev 13828.