update question

[Didrik Pinte]

Hi,

I've made little bugfix and one change to the code. Before commiting, i
need your opinion on it.

Here is the future change log :
-----------------------------------------------------------------------
2006-09-28 Didrik Pinte <dpinte at itae.be>

    * Thuban/UI/classgen.py: bugfix due to wx2.6 update

    * Thuban/UI/view.py: removed non needed declaration

    * Thuban/UI/viewport.py : extracted method from LabelShapeAt to
    GetLabelPosForShape (see feature request #121 on wald)

    * Thuban/UI/selection.py : typo correction
-----------------------------------------------------------------------

[1] Concerning the modification of the Thuban/UI/viewport.py file, i've
just extracted some code of the LabelShapeAt method in order to be used
by a more general method that will allow the user to label a complete
layer by selecting a field of the associated table. See
http://wald.intevation.org/tracker/index.php?func=detail&aid=121&group_id=6&atid=108 
The new method GetLabelPosForShape is at the moment in the viewport.py
file but IMHO, it could be moved to the Thuban/Model/map.py file. What
do you think about this ? Is there an other place that could be more
appropriate ?

[2] What do you think about adding a new tool allowing the user to label
all the shapes of an existing layer (not only one shape like it is
now) ? If the answer is yes, i can easily integrate the work i've just
done into the trunk.

Didrik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Ceci est une partie de message
	=?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=
Url : http://www.intevation.de/pipermail/thuban-devel/attachments/20060928/63e2ba87/attachment.bin