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.

Sign up for GitHub

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

Jump to bottom

Styles: request type colors, map markers/tooltips stacking order #771

Merged
merged 2 commits into from
Aug 5, 2020
Merged

Styles: request type colors, map markers/tooltips stacking order #771

merged 2 commits into from
Aug 5, 2020

Conversation

adamkendis
Copy link
Member

@adamkendis adamkendis commented Aug 5, 2020 •
edited
Loading

  • Changed to primary request type colors for prod update.
  • Fixed stacking order for map markers/clusters and tooltips.

Tooltip above markers unless marker/cluster is hovered:

Screen Shot 2020-08-05 at 11 10 08 AM

Markers/clusters above NC name at zoom:

Screen Shot 2020-08-05 at 11 10 40 AM

  • Up to date with dev branch
  • Branch name follows guidelines
  • All PR Status checks are successful
  • Peer reviewed and approved

Any questions? See the getting started guide

@adamkendis adamkendis requested a review from jmensch1 August 5, 2020 18:14
@adamkendis adamkendis modified the milestone: 311 Data - Go Public Aug 5, 2020
jmensch1
jmensch1 approved these changes Aug 5, 2020
View reviewed changes
Copy link
Contributor

@jmensch1 jmensch1 left a comment

Choose a reason for hiding this comment

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

looks good

@adamkendis adamkendis merged commit 7be8a19 into hackforla:dev Aug 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmensch1 jmensch1 jmensch1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@adamkendis @jmensch1