You need to log in before you can comment on or make changes to this bug.
When trying to point to a location with the ruler tool, the location clicked is not exactly the location which is being used for measuring. This makes it difficult to exactly measure things. Or maybe this location is used for measuring, but the block image is centered at a different location than the location clicked with the point of the arrow.
agreed ... the ruler image position should be centered on the cursor rather than the top left of the image being used. I know that we fixed this for the ROIPolygonTool widget, perhaps the same fix can be applied back to the ruler widget. To be done for 1.99 if its an easy fix, otherwise we'll look at it later
updated version to 1.99
Changed Target Milestone to 1.99 RC
I did a fix to this by changing the shiftTo function in dhtmlapi.js file to: function shiftTo(obj, x, y) { var theObj = getObject(obj) if (isNav4) { theObj.moveTo(x-4,y-4) } else { theObj.left = x - 4 theObj.top = y - 4 } } Far as I can tell the ruler is the only widget thats accessing dhtmlapi.js so this shouldnt break anything. Ideally I think the ruler widget should be changed to use the same functions as the ROIPolygonTool. The code is much cleaner and easier to follow and since many of the things being done are identical there isnt a point in having two completely different methods. Ken
Ken, you are absolutely correct and that has been one of the things on my list for some time. The ruler widget was developed with contributed code very early on, before we really gained a lot of knowledge about javascript and dhtml. The ROIPolygonTool was a complete re-write of that logic, but it has not yet been ported back to the ruler tool. I would actually like to extract all the code for various dhtml drawing routines into one consolidated api and have the various widgets 'use' the drawing tools. But that will not happen in this release unfortunately.
Ruler is on my hitlist for a complete overhaul, I've seen much better implementations than ours.
marking as wontfix because I want any time spent on the ruler widget to go into reimplementing it. see bug 923