[GitHub] zeppelin issue #1775: [ZEPPELIN-1745] View revisions in non-editable mode

[1ambda]

Github user 1ambda commented on the issue:

    https://github.com/apache/zeppelin/pull/1775
  
    Works well as described. Here are few things to consider.
    
    - It would be great if there is a signal (whatever) so let user know that 
this is committed version of note. It's confusing at first for me. 
    - Enabling and modifying committed notes affects on the HEAD note. What 
about disabling **double click**  to committed paragraphs (you can check this 
in screenshot below)
    
    
![pr1775](https://cloud.githubusercontent.com/assets/4968473/21451171/4a51ab30-c940-11e6-955e-1ca3a2f266ac.gif)
    
    - the last one is not directly related with this issue. But It would be 
good to hide git notebook related icons if we are not using `GitNotebookRepo`
    
    <img width="398" alt="screen shot 2016-12-23 at 6 50 35 pm" 
src="https://cloud.githubusercontent.com/assets/4968473/21451287/4a921aa2-c941-11e6-9dae-a56770ce06cf.png";>
    
    Thanks for implementing great feature.


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