[GitHub] zeppelin issue #1210: [ZEPPELIN-1220] Add geographical map as visualization ...

[mwpenny]

Github user mwpenny commented on the issue:

    https://github.com/apache/zeppelin/pull/1210
  
    @corneadoug I'm all to familiar with that magic when trying to reproduce 
bugs ;)
    
    I've made the changes.
    
    * __Small popup:__ The side and bottom margins of the popup are present due 
to yesterday's changes (differentiating the white of the popup from the white 
of the paragraph) and the extra-large top margin is from Esri's CSS. After 
decreasing the top margin the popup is still very cluttered but a bit more 
usable.
    
![zeppelin13](https://cloud.githubusercontent.com/assets/9356790/17601232/0d439994-5fd5-11e6-89fa-bd8fbb64d960.png)
    Even with some extra CSS styling, I just don't think there's enough space 
to display the popup nicely in a paragraph that small. Other graph modes have a 
similar problem which users are likely already accustomed to:
    
![zeppelin14](https://cloud.githubusercontent.com/assets/9356790/17601551/b36a6ff4-5fd6-11e6-99af-858f4eb72347.png)
    
    
    
    * __Internet error message:__ I went with a look similar to the other graph 
modes:
    
![zeppelin12](https://cloud.githubusercontent.com/assets/9356790/17600368/2353e076-5fd1-11e6-8d29-20ca4001c169.png)
    
    * __Lazy loading:__ Similar to the way esriLoader loads the map JavaScript, 
the CSS is now lazy loaded the first time it is needed.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---