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

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

    https://github.com/apache/zeppelin/pull/1730
  
    @tae-jun Nice feature indeed!
    Most of cases are working like a charm. 
    But as @marchpig said above, 
    ![screen shot 2016-12-14 at 12 28 46 
pm](https://cloud.githubusercontent.com/assets/10060731/21168719/bff892d4-c1f9-11e6-8644-9ced7a81d955.png)
    is this intended result? I removed the `note1` and `note2` separately, and 
the time stamped only for latter one: `NoteDirB/note2`.
    
    And another really minor thing is
    ![screen shot 2016-12-14 at 12 41 20 
pm](https://cloud.githubusercontent.com/assets/10060731/21168885/dd423fb0-c1fa-11e6-8b7f-2a5337497a79.png)
    
    As you can see there are two trash can icons and their sizes are different 
now. Hmm.. so I would suggest you to use another icon for `Empty trash`(not 
sure which one will be proper for this..kk) since the trash can icon is already 
used for `Move note to trash` / `Move folder to trash` / `Remove note 
permanently` / `Remove folder permanently`.  



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