Fix issue 579

[daniel2101]

• Please check if the PR fulfills these requirements

• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

• What kind of change does this PR introduce? Bug fix

• What is the current behavior? Issue when we mix Rotate with BrushTool, EllipseRoiTool and RectangleRoiTool. #579

• What is the new behavior (if this is a feature change)? Bug fix, correct rotation
  
  

• Does this PR introduce a breaking change? No

• Other information:

[dannyrb]

Thanks for this! It looks like you have some failing unit tests. If you could resolve them, I would appreciate it.

[JamesAPetts]

Nice! This is packaged up in a really clean way, I like it. Like Danny says, if you could update the unit tests that'd be awesome. Then I think we're golden and this gets a thumbs up from me 👍 .

[daniel2101]

Thanks for your comments. I fix unit tests and try again.

[JamesAPetts]

Thanks for this @daniel2101 ! Added one commit so this change will cut a new release.

[codecov]

Codecov Report

Merging #916 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #916      +/-   ##
==========================================
- Coverage   11.78%   11.73%   -0.05%     
==========================================
  Files         218      218              
  Lines        7129     7156      +27     
  Branches     1120     1124       +4     
==========================================
  Hits          840      840              
- Misses       5318     5341      +23     
- Partials      971      975       +4

Impacted Files	Coverage Δ
src/tools/annotation/RectangleRoiTool.js	16.52% <0%> (ø)	⬆️
src/util/pointProjector.js	0% <0%> (ø)	⬆️
src/drawing/drawEllipse.js	0% <0%> (ø)	⬆️
src/tools/annotation/EllipticalRoiTool.js	18.86% <0%> (ø)	⬆️
src/drawing/drawRect.js	0% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c347ab...b1fb8e7. Read the comment docs.

[codecov]

Codecov Report

Merging #916 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #916      +/-   ##
==========================================
- Coverage   11.78%   11.73%   -0.05%     
==========================================
  Files         218      218              
  Lines        7129     7156      +27     
  Branches     1120     1124       +4     
==========================================
  Hits          840      840              
- Misses       5318     5341      +23     
- Partials      971      975       +4

Impacted Files	Coverage Δ
src/tools/annotation/RectangleRoiTool.js	16.52% <0%> (ø)	⬆️
src/util/pointProjector.js	0% <0%> (ø)	⬆️
src/drawing/drawEllipse.js	0% <0%> (ø)	⬆️
src/tools/annotation/EllipticalRoiTool.js	18.86% <0%> (ø)	⬆️
src/drawing/drawRect.js	0% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c347ab...b1fb8e7. Read the comment docs.

[dannyrb]

🎉 This PR is included in version 3.7.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[dannyrb]

@all-contributors please add @daniel2101 for this cool code contribution!

[allcontributors]

@dannyrb

I've put up a pull request to add @daniel2101! 🎉