cornerstonejs / cornerstoneTools / 567

7%

push

travis-ci

feat(freehandROI) Revamped freehand pencil mode and added handle editing (#402)

* Introduced and refactored new libraries to fork of master

* Introduced new add/remove functionality for freehand tool

* Refactored handleData to a class to ease maintainability now it is introduced in more than one location.

* Fixed filenaming issue of freehandIntersect.js and freehandArea.js

* reverting file names of freeHandArea and freeHandIntersect

* Addressing codeclimate issues

* Addressing code climate issues

* minimum distance for pencilMode and UI improvements

* Working 'chain link' UI feature, still minor visual bug until one moves the mouse

* Added config flag to show all handles of freehand tool all the time

* Fixd bug where the chainlink would display on previously drawn polygons, whilst drawing a new one

* Cleaning up the formating of some code

* Addressing code climate complexity issues

* Added jsdocs to all freehand functions and files

* Added test for freeHandArea

* Added tests for freeHandIntersect.js, calculateFreehandStatistics.js and pointInFreehand.js

* Fixed a bug with ctrl + click removal when clicking on the origin node

* Refactored freehand.pointNearHandle to return null if it found no nearby handle, and propograted this through. This is more intuitive and removes a load of boilerplate elsewhere, considring a valid handle can be 0, i.e. falsy

* Fixed bug where one could self intersect the last line upon completion in polygon mode (thanks @diego0020)

* Updated freehand test page (note, incomplete)

* Added tabindex = 0 and forced focus on the element of freehandRoi test

* Added support for CTRL, SHIFT and ALT/OPTION modifiers. Holding CTRL shows all points on an ROI, making editing easier.

* Refactored freehand's keyDownCallback and keyUpCallback to utils/freehand/keysHeld.js

* Expanding instructions of freehandROI example

* Removed author tags from util/freehand and removed ... (continued)

104 of 2387 branches covered (4.36%)

Branch coverage included in aggregate %.

397 of 5905 relevant lines covered (6.72%)

2.89 hits per line