GitHub user soralee reopened a pull request:

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

    [ZEPPELIN-2396] eliminate the 'ctrl/command+L' keyboard shortcut

    ### What is this PR for?
    This PR is for that eliminate the `Ctrl/Command+L` shortcut keyboard. This 
function is what the ace editor defined.
    
    ### What type of PR is it?
    [Improvement | Document]
    
    ### What is the Jira issue?
    * [ZEPPELIN-2396](https://issues.apache.org/jira/browse/ZEPPELIN-2396)
    
    ### How should this be tested?
    * **Improvement** - Press `Ctrl + L` key and check if showing the dialog or 
not
    * **Document** - Run document development mode and check [this 
document](http://localhost:4000/install/upgrade.html#upgrading-from-zeppelin-07-to-08)
    
    ### Screenshots (if appropriate)
    [Before]
    
![image](https://cloud.githubusercontent.com/assets/8110458/24990819/8529312a-2051-11e7-8849-e00803310752.png)
    
    [After]
    
![z_zeppelin-2396](https://cloud.githubusercontent.com/assets/8110458/24990966/9764e108-2052-11e7-9387-560f9d587782.gif)
    
    
    
    [Document]
    
![image](https://cloud.githubusercontent.com/assets/8110458/25040847/e772e024-2146-11e7-9ded-322c589b424b.png)
    
    
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? Yes, I did. Please review [this 
document](http://localhost:4000/install/upgrade.html#upgrading-from-zeppelin-07-to-08)
 if you run document development mode (localhost:4000)


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

    $ git pull https://github.com/soralee/zeppelin ZEPPELIN-2396_ctrl_l

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

    https://github.com/apache/zeppelin/pull/2248.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 #2248
    
----
commit 4e1b8465a84099efff077e6767d643ee148a4fad
Author: soralee <sora0...@zepl.com>
Date:   2017-04-13T04:49:11Z

    ZEPPELIN-2396 drop the Ctrl+L shortcut keyboard

commit 5f30fe473b6b893895d038d5d5a3468820c4ad2d
Author: soralee <sora0...@zepl.com>
Date:   2017-04-14T10:08:34Z

    upgrade eliminate keyboard shortcut feature and add document

commit 928f179821642c7ccb4fda5deec6b778322cbd6f
Author: soralee <sora0...@zepl.com>
Date:   2017-04-15T04:49:04Z

    add space between + and who is providing

commit d1f966a36c759a87398d65284afb09d1d8f7bb48
Author: soralee <sora0...@zepl.com>
Date:   2017-04-15T05:00:42Z

    modify document

commit bb0f7d8223d6516e037215c552fcd62792accfb3
Author: soralee <sora0...@zepl.com>
Date:   2017-04-15T05:01:40Z

    modify document

----


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

Reply via email to