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

Task/DES-2006: Add ability to display custom styles and icons on map. #65

Merged
merged 5 commits into from
Jul 8, 2021

Conversation

duckonomy
Copy link
Contributor

@duckonomy duckonomy commented Jun 17, 2021

Overview:

Adds ability to detect style properties in a feature and display it on the map.

PR Status:

  • Ready.
  • Work in Progress.
  • Hold.

Related Jira tickets:

Summary of Changes:

Changed leafletUtils.ts to add custom marker creators and changed some lines to remove eslint/tslint warnings.

Testing Steps:

1. Try to put something like the below after line 150 in map.component.ts
2. Check that it loads the colors/icon
3. Also try it without the icon property and change d.properties.customMarker to 'styled'
4. This should show a circle marker with the style applied to it.
Import files under version 2 from https://www.designsafe-ci.org/data/browser/projects/3676202356334531051-242ac117-0001-012/

UI Photos:

Screenshot 2021-06-17 at 17-19-02 HazMapper

Notes:

@duckonomy duckonomy changed the title Add ability to display custom styles and icons on map. Task/DES-2006: Add ability to display custom styles and icons on map. Jun 17, 2021
Copy link
Collaborator

@nathanfranklin nathanfranklin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well. 👍

I made some test files for the current version of this branch https://www.designsafe-ci.org/data/browser/projects/3676202356334531051-242ac117-0001-012/ in version1 subfolder.

I have some suggestions that I'll ask about more specificaly on slack about:

  • support for non-point geometries like polgons
  • form of properties related to this

@reptillicus reptillicus merged commit 1cada3d into master Jul 8, 2021
@taoteg taoteg deleted the task/DES-2006-marker-styles branch July 6, 2023 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants