[GitHub] zeppelin issue #1730: [ZEPPELIN-1736] Introduce trash & enable removing fold...

[tae-jun]

Github user tae-jun commented on the issue:

    https://github.com/apache/zeppelin/pull/1730
  
    @AhyoungRyu Always thanks for the review :) Also thanks for your 
understanding.
    
    @1ambda  Thanks for the review! I appreciate it and sorry for my late 
response.
    That was a really nice point. I fixed it as you see below:
    
    <img 
src="https://cloud.githubusercontent.com/assets/8201019/21429799/b678e794-c8a3-11e6-851f-04d1ad55998e.png";
 alt="image" width=200>
    I tested on Chrome and Firefox. Please check!
    
    
    As to the second point that you mentioned, I thought a lot of about it as 
well :)
    My opinion is same with you, but I think maybe other users can feel limited.
    So I would like to make it configurable as three levels on other PR:
    1. super strict: not even open note
    2. read only: only can open note and read it
    3. unlimited access: able to rename, read, write, and run note
    
    Would you mind if I do that? Please let me know 😄
    Thanks for the review again.


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