List of suggestions #3

[ProjectionWizard]

• [x] Setting to have “round” numbers for central meridians. For example, when I create a map of Africa, the central meridian is 17º 55' 47'’ E. I would prefer 18º, what is probably what most people would use, because then you end up with a straight central meridian line on the map. (done in https://github.com/ProjectionWizard/projectionwizard.github.io/commit/fc31a569e37b2d8d276aa45c02f72eb32145c65e)
• [x] For the equidistant world maps, it would be nice to have the option to enter the central point. (done in https://github.com/ProjectionWizard/projectionwizard.github.io/commit/f7b7a5efcc565a8f84770b478ecb07f6676b8b74)
• [x] Maybe reduce the size of the title a little bit. Maybe convert the title to all uppercase. (done in https://github.com/ProjectionWizard/projectionwizard.github.io/commit/9453f40c9900687c550338409da8bf8cb8708d60)
• [ ] When the entire globe is selected, and the selection is then moved, then the selection shrinks vertically. It probably shouldn't shrink. (Select Entire World button is one time selection that sets the rectangle size to the full extent. User cannot toggle this option "on" and "off." Any next interaction by the user is considered as change of the rectangle)
• [x] In the screenshot below, only the pole should be shown. This is related to previous item. Rectangle visually shrank, but it still reads as whole world selection. (done in https://github.com/ProjectionWizard/projectionwizard.github.io/commit/4a078d52171df31e7617b2f3207ce4f28fec4445 and https://github.com/ProjectionWizard/projectionwizard.github.io/commit/454b84406638fafb7a57ce095ed2f20279b98325)
• [x] The "active-projection" CSS class is not being added to the projection info when there is only 1 projection available for certain projection recommendation conditions. Check current usage of the function highlightActiveProjectionNode. (done in https://github.com/ProjectionWizard/projectionwizard.github.io/commit/e252993516343de3d10a391e64f5dad944c72b2a)

[jwasilgeo]

Work on this has begun in branch 3-suggestions.

[jwasilgeo]

See commit https://github.com/ProjectionWizard/projectionwizard.github.io/commit/4a078d52171df31e7617b2f3207ce4f28fec4445 which prevents a rectangle from being dragged too far north or south. This relates to the last item in the checklist above.

[beuan]

I just pushed several commits to branch 3-suggestions. It includes fixes for the first, third, and last item in the checklist above. There are also some other additional fixes in this branch.

[beuan]

I added sliders to fix the second item in the checklist. @jwasilgeo, any opinions?

[jwasilgeo]

The sliders are a neat addition. What jquery ui lib are they coming from? I ask because the handle sizes a different and bigger here: https://jqueryui.com/slider/

[beuan]

Yes, sliders are from this jQuery lib. I modified their sizes to make them smaller. Thanks for fixing callback function. 👌 https://github.com/ProjectionWizard/projectionwizard.github.io/blob/d21a0f77a045eb4aa8ef2050f626591eed54f23a/page.css#L231-L237

[beuan]

See commit https://github.com/ProjectionWizard/projectionwizard.github.io/commit/e252993516343de3d10a391e64f5dad944c72b2a for the additional item on the list. @jwasilgeo, can you double check if it works for all cases?

[jwasilgeo]

Hey @beuan, thanks for working on that. I am revisiting the code and realize that what it needs is additional assignments of the variable activeProjection so that the function highlightActiveProjectionNode isn't trying to look up an incorrect DOM node. The var assignment wasn't used in enough places; for example ,when I do a find-all for activeProjection = I get only 3 results among outputFormat.js and previewMap.js. Let me take another look at this.

[jwasilgeo]

Please let me know what you think about commit https://github.com/ProjectionWizard/projectionwizard.github.io/commit/bcef97923e72b1f5ef4d5412a1debb1439b5f420. Same effect, but helps the highlight node function to run in all cases with correct activeProjection variable.

I also noticed some projection title nodes were <b>, so I added font-weight: bold; to the .active-projection css class to make other projection title nodes that are <span> have similar style. Maybe this is too much, if so, feel free to remove of course, Mr. 🗺️ 🧙 .

[beuan]

Having <b> with style font-weight: bold is a bit overkill, so for cases when only one projection is listed I replaced <b> with <span> . See https://github.com/ProjectionWizard/projectionwizard.github.io/commit/814567898f8ddcfb1f06fcfe8d4c3774c53f7637.

[ProjectionWizard]

DONE