[GitHub] zeppelin pull request #1971: When gear icon is clicked, scroll the page to t...

2017-02-07 Thread asfgit
Github user asfgit closed the pull request at:

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


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


[GitHub] zeppelin pull request #1971: When gear icon is clicked, scroll the page to t...

2017-02-03 Thread beriaanirudh
GitHub user beriaanirudh opened a pull request:

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

When gear icon is clicked, scroll the page to the top to show the 
interpreter bindings



### What is this PR for?
When someone is browsing a notebook and they click the *gear* button of 
interpreter bindings, the bindings do come up, but the person might not realize 
since it opens on the top of the page, and they don't get any feedback that the 
bindings have opened. We should scroll up the page to show the bindings. This 
will also act as the feedback of clicking the *gear* button.

### What type of PR is it?
[Improvement]

### Todos
* None

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2047

### How should this be tested?
Create few (say 10) paragraphs and scroll down to the last paragraph. Now 
click the *gear* icon meant to open the interpreter bindings. The view should 
have scrolled up to show the bindings.

### Screenshots (if appropriate)



![zeppelin-2047](https://cloud.githubusercontent.com/assets/4542030/22588623/a77b8c2e-ea2c-11e6-8cac-b9f84a28764b.gif)





### Questions:
* Does the licenses files need update?
No
* Is there breaking changes for older versions?
No
* Does this needs documentation?
No. Its a usability improvement.


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

$ git pull https://github.com/beriaanirudh/zeppelin master

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

https://github.com/apache/zeppelin/pull/1971.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 #1971


commit a0f521af621ca6632d7648f019bf3b71d0be2bca
Author: Beria 
Date:   2017-02-03T08:35:49Z

When gear icon is clicked, scroll the page to the top to show the 
intepreter bindings




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