smas1 / geoext-viewer

Automatically exported from code.google.com/p/geoext-viewer
GNU General Public License v3.0
0 stars 0 forks source link

Enhancements and fixes to measure tools #106

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
From Wolfram.Winter Deutsche Bahn

issue # 07
----------
Meassure tools - need a running length / aerea

status issue # 07: fixed
-----------------
implemented

issue # 08
----------
Meassure tools - need an other style of the cursor when active

status issue # 08: fixed
-----------------
implemented - see issue # 02

issue # 13
----------
Meassure tools - I could not implement showing the last measurement, because 
the routine
.onMeasurements / .onMeasurementsPartial are called twice - Why - see comment 
in the beginning
of ToolbarBuilder.js??

status issue # 13: open
-----------------

Original issue reported on code.google.com by jus...@gmail.com on 3 Jul 2012 at 11:51

GoogleCodeExporter commented 9 years ago
Implemented running length and have fixed Wolfram's issue #13: event callbacks 
were done twice since both area and length measurement init (create()) each 
subscribed to _all_ OL Measure controls. Now callbacks are only called once. I 
have integrated some stuff for "Leg" (Teilstrecke") display but the output 
looks a bit funny. This has to be looked into. All code is in SVN
and example on:

http://lib.heron-mc.org/heron/latest/examples/default

(we can now discuss issues as comments in this issue tracker)

Original comment by jus...@gmail.com on 4 Jul 2012 at 6:48

GoogleCodeExporter commented 9 years ago
Please comment // line 40 and 61 'out += "  (" + __('Leg')   ...' - the wrong 
delta running length isn't shown anymore.
My code works only for a 'STATIC' length measurements - therefore you have to 
comment line 245 in 'ToolbarBuilder.js' instead to: // immediate: true, 
If done, the static length measurement shows the correct difference - but only 
when you do a click. 
My approach was to have a running tool - that works only with a new 'onclick' 
handler to get the 'measureLastLength'.

Original comment by wolfram.winter on 5 Jul 2012 at 4:56

GoogleCodeExporter commented 9 years ago
I hope I understand well and made your proposed changes except the immediate: 
true which seems to work ok. Feel free to modify otherwise.

btw if you hold SHIFT+Mouse drawing is without clicks.

Original comment by jus...@gmail.com on 6 Jul 2012 at 2:43

GoogleCodeExporter commented 9 years ago

Original comment by jus...@gmail.com on 17 Jul 2012 at 12:26