Github user ankurmitujjain commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/714#issuecomment-211921573
  
    @corneadoug I think its not right/user friendly way to have download data 
functionality.
    
     You liked approach of including download buttons in "settings".
    But I believe **"Settings" is something to provide configuration/setting 
options and not to provide any functionality**.
    
    Also @adejanovski 
[#6](https://github.com/apache/incubator-zeppelin/pull/6#issuecomment-185086161)
 commented rightly that 
    
    > viewers have to find out that they need to click on the gear, which is 
hidden in report mode and simply absent when you link directly to the paragraph 
as an iframe
    
    So I believe it's not a good idea to include buttons in settings as many 
community member don't liked it.
    
    Secondly if DataTables provides export functionality which works and is 
extensible, than we should use it instead of writing our own way to do data 
export, this I mentioned earlier as well... 
[#6](https://github.com/apache/incubator-zeppelin/pull/6#issuecomment-185081090)
    
    Also about extra UI space, its not an issue for using datatable or any of 
its functionality, but its general issue and to fix it we need to have a 
different PR...
    
    If I look different notebook implementation, than the way they handle this 
issue is like below...
    They are providing all buttons at bottom of table, and zeppelin provides 
same on top of table.
    
    
![image](https://cloud.githubusercontent.com/assets/1140475/14640501/609d5802-065f-11e6-8bd3-1758181eb14d.png)


---
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.
---

Reply via email to