[GitHub] zeppelin pull request #1665: [ZEPPELIN-1694] Show result when editor languag...

[minahlee]

GitHub user minahlee opened a pull request:

    https://github.com/apache/zeppelin/pull/1665

    [ZEPPELIN-1694] Show result when editor language changed from markup to 
another

    ### What is this PR for?
    * Show result when editor language changed from markup to another
    * Fix syntax highlight bug when paragraph contains String `%something` in 
the middle of contents in case the default repl name(`%spark`) is not specified 
at the beginning of paragraph.
    * Save `editOnDblClick` field in note.json so it can be applied after 
reloading page.
    
    ### What type of PR is it?
    Bug Fix | Improvement
    
    ### What is the Jira issue?
    [ZEPPELIN-1694](https://issues.apache.org/jira/browse/ZEPPELIN-1694)
    
    ### How should this be tested?
    Outline the steps to test the PR here.
    
    ### Screenshots (if appropriate)
    **Show result after repl change**
    Before
    ![nov-21-2016 
17-21-17](https://cloud.githubusercontent.com/assets/8503346/20490686/1479aba6-b00f-11e6-9376-0d3e1df484b1.gif)
    
    After
    ![nov-21-2016 
17-21-28](https://cloud.githubusercontent.com/assets/8503346/20490690/17d1c3d8-b00f-11e6-9d5f-7217b74044ae.gif)
    
    
    **Fix highlight bug**
    Before
    <img width="324" alt="screen shot 2016-11-21 at 5 14 23 pm" 
src="https://cloud.githubusercontent.com/assets/8503346/20490467/694e522c-b00e-11e6-984e-611d0a7ff4a5.png";>
    
    After
    <img width="332" alt="screen shot 2016-11-21 at 5 12 00 pm" 
src="https://cloud.githubusercontent.com/assets/8503346/20490472/6d00ee2a-b00e-11e6-9908-8f7b8a0f42f0.png";>
    
    **Fix edit on double click bug after refresh**
    Before
    ![nov-21-2016 
17-25-13](https://cloud.githubusercontent.com/assets/8503346/20490833/916d7f20-b00f-11e6-992a-7b9946898e30.gif)
    
    After
    ![nov-21-2016 
17-25-44](https://cloud.githubusercontent.com/assets/8503346/20490838/95fe9682-b00f-11e6-96a4-40b67e859267.gif)
    
    ### Questions:
    * Does the licenses files need update? no
    * Is there breaking changes for older versions? no
    * Does this needs documentation? no


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/minahlee/zeppelin ZEPPELIN-1694

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/1665.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1665
    
----
commit 2755b49d13bc048d38f28f548d6388bbf4cf4466
Author: Mina Lee <mina...@apache.org>
Date:   2016-11-21T12:24:50Z

    Fix bug of wrong syntax highlight when there is %html String in paragraph

commit a2d54fcaf6753dbf036312742ba4b440efa15596
Author: Mina Lee <mina...@apache.org>
Date:   2016-11-21T15:59:51Z

    Open output if editor modechanged from markup language to others

----


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