Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Basemap switcher #111

Open
tgilcrest opened this issue May 2, 2020 · 1 comment
Open

Basemap switcher #111

tgilcrest opened this issue May 2, 2020 · 1 comment

Comments

@tgilcrest
Copy link

tgilcrest commented May 2, 2020

We want to add a basemap switcher to the app to give users the ability to view basemaps underneath their maps. This task involves adding a basemap switcher and showing basemaps underneath the geounit lines, population labels, and district shapes. Note that because we won't have a transparency slider, we'll also have to modify how the districts appear (make them transparent) when a basemap is present. We'll use Mapbox basemaps for this. Links to be provided.

AC:

  1. There is a basemap switcher in the top right corner that allows selecting from different basemaps
  • None (default)
  • Streets
  • Satellite
  • Topography
  1. There is a preview thumbnail for each basemap option
  2. When a basemap is selected, the districts colors become transparent, with a less-transparent color border on the boundary of the district.
  3. Unlike the prototype, we should make the unassigned areas transparent (not translucent white). Note that for Mapbox GL, “transparent” is a color, so this could make it good way to drop it in as a color (and replace if we need to make it a non-transparent color).

Basemap selector (don't worry about creating the "Imagery" section for now):
image.png

Transparent districts:
image.png

@jfrankl jfrankl added the design label Sep 2, 2020
@jfrankl jfrankl self-assigned this Sep 9, 2020
@jfrankl jfrankl removed the design label Sep 10, 2020
@jfrankl jfrankl removed their assignment Sep 10, 2020
@jfrankl jfrankl mentioned this issue Sep 10, 2020
@jfrankl
Copy link
Contributor

jfrankl commented Sep 10, 2020

I just opened a pull request that adds a basic basemap to the application #401. That will be suitable for launch.

In terms of adding a basemap switcher, I think we should spend some time thinking through our needs here. Getting the aerial imagery view to work well might be a big challenge, and I think it would be worth confirming its value. Instead of having 2 or 3 basemaps to choose from, we may want to explore an option where the user can choose between aspects of the basemap. For example: toggle street labels, toggle city labels, toggle hillshade.

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

No branches or pull requests

4 participants