search

Esri / distance-direction-addin-dotnet

Add-in provides the ability to easily and quickly create geodesy lines, circles, ellipses and range rings.
Apache License 2.0
17 stars 22 forks source link

Distance and Direction Ring [and ellipse] tool causes ArcMap to go unresponsive click outside basemap extent #535

Closed BobBooth closed 6 years ago

BobBooth commented 6 years ago

ArcMap 10.6.0.8321 Distance and Direction 2.2.0.31 (3/5/18) ArcMap 10.6.0.8321 and MT build 23 (5Mar) When you create a set of rings and radials (non-interactive mode), set a distance, number of rings, number of radials and click outside the extent of the map, add a point, it is not handled, and you can hit enter to start creating range rings, but ArcMap goes unresponsive. This also affects Ellipses. We should handle this like we do Circles (https://github.com/Esri/distance-direction-addin-dotnet/issues/512).

image image image image

csmoore commented 6 years ago

Addressed in #546 - ready for QA/QC

While testing this I saw another, less common issue - written up in a new issue #547

ACueva commented 6 years ago

I have tested this on ArcMap 10.6, the fixes appear to have worked, however, I'm noticing the following:

2018-04-20_13-53-26

The label appears to be upside down while zoomed way out. Once you zoom in, it appears fine. 2018-04-20_14-31-20

Issue logged to address these findings: https://github.com/Esri/distance-direction-addin-dotnet/issues/552

FYI @csmoore @kgonzago

ACueva commented 6 years ago

Remaining work:

ACueva commented 6 years ago

Added this issue to the Distance and Direction Issues addressed section. 2018-04-23_12-16-43

ACueva commented 6 years ago

@dfoll @topowright I believe this can be moved to done. Do we need a Verify tag to verify my CMS changes?

dfoll commented 6 years ago

verified list as an issue addressed. was never listed as a known issue, so nothing to remove there

dfoll commented 6 years ago

@ACueva I forgot to add that this should be added to the What's New page for Defense and the What's New section on the homepage. This can be done by branching from the 6.2 release of the website repo, and merging.

dfoll commented 6 years ago

Verify