add basemap widget

[allisonmuise]

when collecting feedback in remote areas, a combination of aerial imagery and topo maps may be required to find the correct location. Currently the app only supports one basemap.

[allisonmuise]

The UI for this might shift slightly over the course of development, but the functionality will remain the same, so please start there, and we can tweak the UI after I return.

Desktop mode: I suggest starting with the button graphic from Basic Viewer. This button could sit in line with the navigation widgets on the left side of the screen. Clicking the widget would expand a panel (like the search does) containing the basemap options. This panel should be styled to fit in with Reporter. Do not use the same styling from Basic Viewer.

Smartphone mode: Let's try adding the item 'Basemaps' to the burger menu list in between Sign In and Help. Clicking this should open a popover window (like the help) with the basemap options

[CTLocalGovTeam]

@allisonmuise

Here is the screenshot of base map gallery widget in desktop and mobile mode.

Please confirm if this looks good.

[gigaranger]

Hello Allison,

I like it and will it be configurable for the basemap groups to allow own basemaps in the widget?

Thanks,

Kenny

From: CyberTech [mailto:notifications@github.com] Sent: Friday, September 22, 2017 2:39 AM To: Esri/crowdsource-reporter crowdsource-reporter@noreply.github.com Cc: Kenneth Kleewein Kenneth.Kleewein@matsugov.us; Manual manual@noreply.github.com Subject: Re: [Esri/crowdsource-reporter] add basemap widget (#226)

@allisonmuisehttps://github.com/allisonmuise

Here is the screenshot of base map gallery widget in desktop and mobile mode.

[image]https://user-images.githubusercontent.com/2959186/30740698-a5ddf36e-9faf-11e7-8134-7d1d271c3cfd.png

Please confirm if this looks good.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/Esri/crowdsource-reporter/issues/226#issuecomment-331413475, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANAIEIssXjUZPrqQ7ewdTxpH4JgiHs3qks5sk45UgaJpZM4JrZ6o.

[allisonmuise]

@CTLocalGovTeam I like the UI. The window fits well with the rest of the app.

Will the access point for the gallery be the same on desktop and mobile, or is it under the burger? Can the basemap group be configurable at the app level? After choosing a basemap, does the window close automatically, or does the X need to be clicked after selection? Is the currently selected basemap indicated in any way?

cc @ScottOppmann

[CTLocalGovTeam]

@allisonmuise

Please see our response inline.

Will the access point for the gallery be the same on desktop and mobile, or is it under the burger?

We are planning to keep the access points same for mobile and desktop. This way user will always see basemap button irrespective of mobile/desktop mode.

Can the basemap group be configurable at the app level?

Yes. We will add a new configuration parameter for selecting basemap group. By default this will be empty. When no group is configured, app will fetch org's basemap group.

After choosing a basemap, does the window close automatically, or does the X need to be clicked after selection?

Once a basemap is selected the window will be closed automatically.

Is the currently selected basemap indicated in any way?

The basemap gallery widget takes care of it and shows a border for selected basemap.

Please let us know your thoughts.

Also, we are planning to give a quick demo once we are done with the implementation. This will make sure all of us are on same page.

[allisonmuise]

@CTLocalGovTeam Perfect! Sounds good!

[CTLocalGovTeam]

@allisonmuise

As discussed on the status call, we are not able to add group selector for allowing user to configure the basemap group. Till the time we take a final call on configuration panel, we are planning to honor the org's settings.

If useVectorBasemaps key is set to true then we will use vectorBasemapGalleryGroupQuery group in basemap gallery widget. Otherwise we will honor the basemapGalleryGroupQuery for creating basemap gallery.

[allisonmuise]

@CTLocalGovTeam Yes, I haven't forgotten about this :). I'm discussing options with the AGOL team and will send you more info as soon as I have it.

[allisonmuise]

@CTLocalGovTeam Could we make this configurable, and off by default?

[CTLocalGovTeam]

@allisonmuise

We can add a new key showBaseMapGallery in the config file (default value will be false) and show it in configuration panel.

[allisonmuise]

@CTLocalGovTeam Thank you!! I realize I never wrote this into the issue description, so I appreciate you adding this parameter.

[allisonmuise]

verified

[gigaranger]

Hello Allison,

I am excited to see this get resolved for the basemaps to be available in the crowdsource reporter.

Is this configuration option available now for a app that is hosted in ArcGIS Online?

If so where do I make the change in the application?

Thanks,

Kenny Kleewein

GIS SPECIALIST MATANUSKA-SUSITNA BOROUGH INFORMATION TECHNOLOGY 350 E DAHLIA AVE PALMER AK 99645-6411 907-861-8536

From: Allison Muise [mailto:notifications@github.com] Sent: Monday, November 06, 2017 12:28 PM To: Esri/crowdsource-reporter crowdsource-reporter@noreply.github.com Cc: Kenneth Kleewein Kenneth.Kleewein@matsugov.us; Manual manual@noreply.github.com Subject: Re: [Esri/crowdsource-reporter] add basemap widget (#226)

Closed #226https://github.com/Esri/crowdsource-reporter/issues/226.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/Esri/crowdsource-reporter/issues/226#event-1328497681, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANAIEHM5wkkihEkMnr6TY-Hv8M5ehQ4Fks5sz3nbgaJpZM4JrZ6o.

[allisonmuise]

@gigaranger This change will be available following the December release of ArcGIS Online.

[skrishn]

@allisonmuise : test case added. Label removed