Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/2568
  
    Thanks @malayhm for great improvement.
    
    Tried this branch and have few feedbacks.
    
    1. looks like it's got more space between buttons. Is it intended? can we 
keep the same space if there's no special reason?
    
    
![image](https://user-images.githubusercontent.com/1540981/31002012-b8060bf2-a49c-11e7-8186-dacc05c2d32d.png)
    
    
    2. I can not see icon image that brings dropdown menu, like below.
    
![image](https://user-images.githubusercontent.com/1540981/31002049-03bdb46e-a49d-11e7-8571-577ae47ce872.png)
    
    3. Is trash icon make sense for delete?
    
    Because of Zeppelin does not store Paragraph in the trash to recover, it 
may introduce wrong expectation on the feature. 
    
![image](https://user-images.githubusercontent.com/1540981/31002070-2bf584f2-a49d-11e7-9b3c-f3b272f13d85.png)
    I think current 'X' icon is more appropriate in this action. 
    
![image](https://user-images.githubusercontent.com/1540981/31002096-506c761a-a49d-11e7-8d0d-7c9aaefd5af5.png)
    
    what do you think?
    
    
    



---

Reply via email to