bulia byak wrote, On 08/18/09 17:29:
This seems to be a cosmetic bug but it's not:
https://bugs.launchpad.net/inkscape/+bug/415168
because the undo comment depends on 2geom matrix functions:
if (_current_relative_affine.isTranslation()) { sp_document_done(sp_desktop_document(_desktop),SP_VERB_CONTEXT_SELECT, _("Move")); } else if (_current_relative_affine.isScale()) { sp_document_done(sp_desktop_document(_desktop), SP_VERB_CONTEXT_SELECT, _("Scale")); } else if (_current_relative_affine.isRotation()) { sp_document_done(sp_desktop_document(_desktop), SP_VERB_CONTEXT_SELECT, _("Rotate")); } else { sp_document_done(sp_desktop_document(_desktop), SP_VERB_CONTEXT_SELECT, _("Skew")); }
and these are quite fundamental to Inkscape operation - if they are broken I'm afraid to think what else might be broken. Can someone with a good grasp of 2geom or just geometry please look into this? I think this must be fixed before the release.
the 2geom function are correct. But that _current_relative_affine variable does not hold the current relative affine. In function Inkscape::SelTrans::transform it is set to the absolute affine matrix.
Thomas