Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

Reference System Label Settings (which may be MGRS) #61

Open
1 of 2 tasks
joebayles opened this issue Sep 6, 2017 · 12 comments
Open
1 of 2 tasks

Reference System Label Settings (which may be MGRS) #61

joebayles opened this issue Sep 6, 2017 · 12 comments

Comments

@joebayles
Copy link
Contributor

joebayles commented Sep 6, 2017

Update

  • Expose in the configuration dialog, under the Reference System tab.
  • Change the tooltip Update strings.js #95

Original Issue

Grid Zone Designator to be off by default with the option to turn the GZD on under the label settings

Instead of creating dialogs to turn on or off things like the Grid Zone Designator and the 100Km Grid Square Identifier, should we just reuse the coordinate notation dialog from the Coordinate Conversion Tool? It has the same look and feel as other widgets, and gives full customization to the user. See below for examples:

Coordinate Conversion Dialog:
image

Options Dialog:
image

@joebayles
Copy link
Contributor Author

Allowing someone to selectively remove GZDs and GSIDs, but select that grid size, would create a situation where they create a label without any text. In subsequent releases, it may be prudent to program the widget to catch that.

@adgiles
Copy link
Contributor

adgiles commented Sep 19, 2017

Addressed in this commit: 53248a9

untitled

@joebayles joebayles self-assigned this Sep 19, 2017
@joebayles
Copy link
Contributor Author

Working great! A few comments:

  1. I think this should also be exposed in the configuration dialog, under the Reference System tab.
  2. I would change the tooltip text below:

Z: Grid Zone Designation (GZD)
S: 100,000-meter grid square designator
X: X Coordinate (Easting)
Y: Y Coordinate (Northing)

Example:
Z-S-X-Y 15S-WC-80817-51205

... to:

Z: Grid Zone Designator (GZD)
S: 100,000-meter Grid Square Identification (GSID)
X: Easting (X Coordinate)
Y: Northing (Y Coordinate)
Examples:
ZSXY is 15SWC8081751205
ZS X,Y is 15SWC 80817,51205

joebayles added a commit that referenced this issue Sep 20, 2017
@joebayles joebayles assigned adgiles and unassigned joebayles Sep 20, 2017
@adgiles
Copy link
Contributor

adgiles commented Sep 21, 2017

@joebayles how do you see the configuration working for this? Do you think we need three input boxes for each of the following formats: Grid Zone, 100,000 and then 10,000 and below?

@adgiles
Copy link
Contributor

adgiles commented Sep 22, 2017

Any thoughts, anyone?

@elinz
Copy link

elinz commented Sep 22, 2017

@adgiles @joebayles If this control is going to be placed on the configuration dialog then I see this as being the same single input textbox that is on the tool dialog; I don't see a reason why we would change the user experience just because we may add it to the configuration dialog.

@adgiles
Copy link
Contributor

adgiles commented Sep 22, 2017

@elinz the content in the text box changes depending on what grid scale you are using:

  • Grid Zone = Z
  • 100k = ZS
  • <100K = ZSXY
    How do we want to handle the three different inputs?

@joebayles
Copy link
Contributor Author

@adgiles, sorry I was out. I think for the first release, using the existing label format dialog would be sufficient. We can explore breaking them out in subsequent releases.

@adgiles
Copy link
Contributor

adgiles commented Sep 25, 2017

This should have been taken out of this sprint last week

@joebayles
Copy link
Contributor Author

Well, the tooltip is updated. Can you not expose that dialog on the configuration pane? If not, then yeah, let's move it to after next.

@adgiles
Copy link
Contributor

adgiles commented Sep 25, 2017

It was removed from this sprint as the PEs said they would not have time to test any implementation of the configuration settings

@joebayles
Copy link
Contributor Author

It's still in the sprint. I'll remove it.

FYI @ArcGIS/arcgis-defense-solutions

@joebayles joebayles modified the milestones: Version - Next, Version - After Next Sep 25, 2017
@lfunkhouser lfunkhouser removed this from the Version - Next milestone Dec 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants