Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/714#issuecomment-211799196
It was already discussed there.
Regarding the TODO, there is only one that is useful: `Hide/disable the
button if it is not in table/chart view`, but it can always be done later. (not
critical for now)
The icon is fine, the Safari problem is not related to this feature (it
affect also the existing export notebook), the test with large dataset doesn't
matter since if it was too big, the browser will crash.
If we get this done with, then we can work on improving it, and build
around it.
Otherwise it just become a blocker
---
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.
---
