search

Siqister / city-x-city

MIT License
1 stars 3 forks source link

Asset icons for different asset types #16

Open allthesignals opened 8 years ago

allthesignals commented 8 years ago

Specifically, the icons used for assets that are plotted. Revisions to asset names to be sent, e.g., "Arts and Rec" to "Rec".

allthesignals commented 8 years ago

Relevant source: https://github.com/Siqister/cartodb_test_app/blob/da42d434bae5ccbe8764be01349e43516474c071/public/js/app/views/mapView.js#L423

A better approach might be to vectorize the icons, load them in CSS, and have map view assign the asset type names as L.icon HTML classes.

IanJakus commented 8 years ago

Add use type an sub types as discussed and outlined on spreadsheet

nupoormonani commented 8 years ago
IanJakus commented 8 years ago

2016-06-27 tdi map icon_bank

2016-06-27 tdi map icon_placemaking

allthesignals commented 8 years ago

Which is the temporary icon. I couldn't find it. Thanks. @IanJakus @nupoormonani

IanJakus commented 8 years ago

the two files that I attached, one is the revised Bank icon, the green one that looks like a farmers market tent is the revised temporary one. @allthesignals

allthesignals commented 8 years ago

@IanJakus Currently, Bank is a subtype, and icons aren't specific to subtypes only types.

IanJakus commented 8 years ago

The icons look great now. Where we have identified icons for sub types we have deemed that those subtypes warrant their own icon. Is it straightforward to add those icons in? @allthesignals

allthesignals commented 8 years ago

@IanJakus Not straightforward at this point. I will look into it more. How should I prioritize that given the current deadline? Thanks!