search

Code4SA / informal-settlements

0 stars 0 forks source link

Map difficult to read #3

Open adieyal opened 8 years ago

adieyal commented 8 years ago

Why black? Also, there are so many boxes and banners in the map, I can barely see the map itself. Why do we need the Code4SA academy logo there? Do we really need to be able to add and remove layers?

Does this really need to be a dynamic map? Wouldn't a stylised static map be clearer?

lailahr commented 8 years ago

Good morning Adi,

I agree with the black and the fact that the banners really do take up a lot of space. At the end of the day Friday I thought about making some changes but was also aware of the urgency of stepping away and just getting the article up. I'm grateful for your input and have made the changes according to your recommendation.

Changes made:

adieyal commented 8 years ago

Regarding branding, I prefer not to compromise the final product. Analytics are important though

Adi On 29 Mar 2016 09:36, "lailahr" notifications@github.com wrote:

Good morning Adi,

I agree with the black and the fact that the banners really do take up a lot of space. At the end of the day Friday I thought about making some changes but was also aware of the urgency of stepping away and just getting the article up. I'm grateful for your input and have made the changes according to your recommendation.

Changes made:

  • switch from black to grey base map;
  • removal of layer selector
  • removal of code4sa banner *however Sarah queried whether if someone did by chance decide to share the map, the visualisation wouldn't be branded?

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/Code4SA/informal-settlements/issues/3#issuecomment-202759540