Hi Hans.
On 10/04/2011 04:02 PM, Hans Breuer wrote:
At 27.09.2011 01:43, Paul Chavent wrote:
Hi dia ml
Here is the last patch for the sozi feature.
There should be no more problems for applying the patches,
Unfortunately the problems in applying the patch(es) persisted.
In fact I was only trying to apply the "Add sozi front end." part.
The other two need some discussion/adaption before being acceptable.
and i had some minor fixes.
With the above mentioned problems I looked at the diff of your patches and noticed you
switched the transformation algorithm to "no apply scale to transformation".
This change I would like to understand: is this a choice the user may want to make?
In fact, the three implementations should result in the same positioning for
the frame. But the sozi player engine seems to not deal well with some
transformation formats. In the original patch, the size of the frame is not
respected... My change should improve this, but you are wright, it's still
buggy.
I will look more in depth the sozi player.
I've created a patch for me to apply to my local version with:
git diff 17004bab c4632ef9 > dia-sozi-2.diff
After adding the necessary bits to build on win32 I've once more played with
your patch by modifying render-test.dia
- only after reverting the agorithm switch (see above) the result
of rotated sozi frame is correct
OK, i will try to find if there can't be some improvement at the sozi player
level as well...
- IMO the use of in-diagram-editing (SoziFrame::title) is
counter-productive:
a) the automatic selection for text deiting complicates the
intial sizing of the shape
b) I was expecting the number to be used as a sequence number, but
both are independent. So I had to change the sequence number, too.
c) the text is a bit obtrusive and there is no way to change it's
font, size or color
I agree with you, moreover, it could avoid the first patch on widgets...
I don't know wich text to display (the frame sequence or the frame title), and
how to display it (i would like to attach it to the top left corner as it is
currently done).
- IMO it would be more user friendly to have the rotation angle in
degrees not radians.
Yes, but in the background it's easier to have radian. I can add a variable to
have the front end in degree and the backend in radians...
- the position you choose for the sozi frame is the center, why not
name it that, too?
What do you mean by "name it" ? I could display the red dot as others shape
anyway...
- the serialization of the SoziFrame::title does not work
OK, i will look for that.
All in all I think your feature is good to be included in Dia master,
but that depends a bit on your future development ideas. Do you plan to
incorporate some of the ideas discussed on the mailing list, like:
- better automatic sequence number generation [1]
Yes definitively.
- restricting the sozi impact to the SVG exporter [2]
(currently it is also emitted to the .shape export)
I think that sozi have only interest in svg. But i would like to find a way to
allow the export to others svg renderers... I haven't think how yet.
Or do you want some more help with it?
I will certainly need help, and i will ask to the list.
Thanks,
Hans
[1] http://mail.gnome.org/archives/dia-list/2011-August/msg00048.html
[2] http://mail.gnome.org/archives/dia-list/2011-September/msg00174.html
-------- Hans "at" Breuer "dot" Org -----------
Tell me what you need, and I'll tell you how to
get along without it. -- Dilbert
_______________________________________________
dia-list mailing list
dia-list@gnome.org
http://mail.gnome.org/mailman/listinfo/dia-list
FAQ at http://live.gnome.org/Dia/Faq
Main page at http://live.gnome.org/Dia
_______________________________________________
dia-list mailing list
dia-list@gnome.org
http://mail.gnome.org/mailman/listinfo/dia-list
FAQ at http://live.gnome.org/Dia/Faq
Main page at http://live.gnome.org/Dia
