Skip to content

Conversation

@Vac1911
Copy link

@Vac1911 Vac1911 commented Mar 25, 2021

Added TACGRP.TSK.CNT / CONTAIN Graphic (GTJ-----****X).
Also added quick geometry function to test if a set of points are clockwise.

I am opening a pull request to your fork (rather than the main repo) as I plan to create boundary/line based contributions in the future. This is just me getting my feet wet in this project.

Screen shot of added graphic (example data included in /example-data/tacticaljson/contain.json):
image

qis-andrew added 2 commits March 25, 2021 15:50
Also added quick geometry function to test if a set of points are clockwise
Originally tried using a line dash styling option (ol/Style/Stroke/setLineDash), however it looked weird when zooming because the dash lengths remained constant as the shape grew and shrank.
@Vac1911
Copy link
Author

Vac1911 commented Mar 26, 2021

I didn't realize the "Added Counterattack" commit was going to be added to the pull request because it was made after the pull request was created but, here is an image of that:
image

Copy link
Member

@ondrej-podolsky ondrej-podolsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you for solving this!

@ondrej-podolsky ondrej-podolsky merged commit 18c55d3 into Military-Tactical-Graphics:master Mar 26, 2021
@Plavit
Copy link
Member

Plavit commented Mar 27, 2021

Great work @Vac1911 , thanks for contributing!

@Vac1911
Copy link
Author

Vac1911 commented Mar 29, 2021

No problem :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants