Code review comment for lp:~inkscape.dev/inkscape/doc_rotate

Revision history for this message
Martin Owens (doctormo) wrote :

The namespace should be saved somewhere instead of re-specified each time:

DOCROTATE = "{http://www.inkscape.org/namespaces/inkscape}document_rotation"

for example.

Everything else looks remarkably good for such a large diff.

review: Approve

« Back to merge proposal