-
Notifications
You must be signed in to change notification settings - Fork 24
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
Pin Logos #25
Comments
In the short term, thoughts on pin color? I'd color code as follows:
We should be able to do standard pins in straight CSS, unless people feel strongly about generating raster image assets for it? |
Those work for me. Just need to make sure it matches up to whatever we use for #37. I think image icons would be a neat feature to have, but we can add them in later if someone gets the energy. For posterity, here's how we'll implement the icons in the future: http://leafletjs.com/examples/custom-icons/. |
So preferring |
It might be simpler, but we can cross that bridge when we revisit it, too. Docs on |
Those would be dope for now |
@vonbearshark could you assign me to this issue? I'm working on at least some basic flat-ui icons. |
For sure! Thanks, @brlodi! Let's nest it in an Edit: looks like I can't assign you--could you join the PittCSC org as a member? |
@vonbearshark You'd need to invite me. There's no option for non-members to request membership |
Of course there's not. That would be simple. Just sent you an inv |
We should delete |
Should be auto-removed once the files in |
Yeah, that's what I mean |
We need to distinguish the pins based on the data coming in. We need to have at least separate icons/colors for the city 311 and police data, for now. But we also need icons for the campus police data, fire data, health and safety restaurant inspection data, etc.
The text was updated successfully, but these errors were encountered: