You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by krskoondra <gi...@git.apache.org> on 2016/11/27 20:27:20 UTC

[GitHub] zeppelin pull request #1688: ZEPPELIN-1637 - Add button for "create next blo...

GitHub user krskoondra opened a pull request:

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

    ZEPPELIN-1637 - Add button for "create next block as copy of this block"

    ### What is this PR for?
    Added new buttton "Copy paragraph" near run button. This new button create copy of current paragraph and insert it below current paragraph.
    
    ### What type of PR is it?
    Improvement
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-1637
    
    ### How should this be tested?
    Write something into paragraph and click on button "Copy paragraph".
    
    ### Screenshots (if appropriate)
    
    ### 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/krskoondra/zeppelin iss1637

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

    https://github.com/apache/zeppelin/pull/1688.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 #1688
    
----
commit f6716e00fd2f3d657ccd4c46df34886401468f48
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-11-27T19:17:33Z

    ZEPPELIN-1637

commit 51165b89a1a6d1bcdbd16b50d94307b96835780a
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-11-27T20:08:37Z

    fix

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by krskoondra <gi...@git.apache.org>.
Github user krskoondra commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    OK, @Leemoonsoo, the button is now in the dropdown menu of the paragraph and new keyboard shortcut ctrl+shift+c is added.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by krskoondra <gi...@git.apache.org>.
Github user krskoondra commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    @Leemoonsoo my id is krskoondra.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    @krskoondra Thanks for the nice feature.
    Merge to master if there're no further discussions.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by krskoondra <gi...@git.apache.org>.
Github user krskoondra commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    Someone help me. How can I resolve "This branch has conflicts that must be resolved"?
    I'm beginner with opensorce and github.
    Thanks.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    @krskoondra Could you let me know you apache JIRA id? 
    so i can assign you in this https://issues.apache.org/jira/browse/ZEPPELIN-1637 issue and resolve.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    Thanks for updatding description! 
    Agree with @Leemoonsoo. And FYI, you can refer #1686 for assigning keyboard shortcut :D


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    Thanks @krskoondra for the contribution.
    LGTM.
    
    merge to master if there're no more discussions.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #1688: [ZEPPELIN-1637] Add button for "create next blo...

Posted by krskoondra <gi...@git.apache.org>.
Github user krskoondra closed the pull request at:

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


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    @krskoondra Thanks for your contribution.
    If there are UI changes, some screenshot images can help other ppl to quickly look into this :)


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by epahomov <gi...@git.apache.org>.
Github user epahomov commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    Yeaaa, I love it!


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    If i assume your remote name for Zeppelin repository is `apache-github` (you can check it by running `git remote -v`), 
    
    First, fetch Zeppelin repo (to get latest master) with `git fetch apache-github`, and then rebase or merge the master branch to your branch `git rebase apache-github/master` or `git merge apache-github/master`. Then git will ask you resolve conflict. After resolve all the conflicts you can push the new commit in this branch.



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #1688: [ZEPPELIN-1637] Add button for "create next blo...

Posted by krskoondra <gi...@git.apache.org>.
GitHub user krskoondra reopened a pull request:

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

    [ZEPPELIN-1637] Add button for "create next block as copy of this block"

    ### What is this PR for?
    Added new buttton "Copy paragraph" near run button. This new button create copy of current paragraph and insert it below current paragraph. For this function is also added new keyboard shortcut ctrl+shift+c.
    
    ### What type of PR is it?
    Improvement
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-1637
    
    ### How should this be tested?
    Write something into paragraph and click on button "Copy paragraph".
    
    ### Screenshots (if appropriate)
    ![screanshot](https://cloud.githubusercontent.com/assets/24210886/20679138/fb0e06cc-b599-11e6-83cb-f3775b0107fc.jpg)
    Click on button \u201cClone this paragraph\u201d to create copy of this paragraph and insert it below this paragraph with opened editor:
    ![screanshot 2](https://cloud.githubusercontent.com/assets/24210886/20679150/073b08d2-b59a-11e6-9250-b6873cbbe9d9.jpg)
    And keyboard shortcuts:
    ![screanshot shortcut](https://cloud.githubusercontent.com/assets/24210886/20679173/174f68c6-b59a-11e6-85bd-39f614456b05.jpg)
    
    
    ### 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/krskoondra/zeppelin iss1637

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

    https://github.com/apache/zeppelin/pull/1688.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 #1688
    
----
commit f6716e00fd2f3d657ccd4c46df34886401468f48
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-11-27T19:17:33Z

    ZEPPELIN-1637

commit 51165b89a1a6d1bcdbd16b50d94307b96835780a
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-11-27T20:08:37Z

    fix

commit 65f30c0fa0dd612d19bdaab0b4b92fc38b40d054
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-11-27T20:32:49Z

    fix

commit 908560ffef0262b90fdca8b108379874b6f851eb
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-11-27T21:34:29Z

    fix

commit ea4e030a1c9c0d6e4d6baa9ef36885bf3d9ec5f1
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-11-27T21:50:46Z

    fix

commit 71751925f4dad6fb89fed6af3f4f00e470c10697
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-11-28T16:57:45Z

    moved butten and added short cut

commit 552d7a1cd405ac52f645136894cf97e718724d66
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-12-01T12:43:48Z

    remove withspace

commit 3a27a69f81d15ca6a373c21af245d45fce24a5f9
Author: Ond\u0159ej Kr\u0161ko <kr...@fel.cvut.cz>
Date:   2016-12-01T13:03:12Z

    fix conflicts

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin pull request #1688: [ZEPPELIN-1637] Add button for "create next blo...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    CI test failed with 
    ```
    Results :
    
    Tests in error: 
      ParagraphActionsIT.testRemoveButton:155->AbstractZeppelinIT.clickAndWait:131 � ElementNotVisible
      ParagraphActionsIT.testMoveUpAndDown:200->AbstractZeppelinIT.clickAndWait:131 � ElementNotVisible
      ParagraphActionsIT.testCreateNewButton:85 � ElementNotVisible Element is not c...
      ZeppelinIT.testAngularDisplay:157->AbstractZeppelinIT.waitForText:84 � StaleElementReference
    
    Tests run: 18, Failures: 0, Errors: 4, Skipped: 0
    ```
    screenshot from ci server
    ![image](https://cloud.githubusercontent.com/assets/1540981/20824096/7f486f60-b80e-11e6-9e35-64985df71bb1.png)
    
    I don't think this PR caused the error but to make sure, could you trigger CI build again?
    You can add commit or close/open this pullrequest to trigger CI build.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1688: [ZEPPELIN-1637] Add button for "create next block as c...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/1688
  
    Top right corner of each paragraph was pretty crowded in early days of Zeppelin.
    Run, Move up, Move down, Delete, Show title, Show codes, Hides output, and so on.
    
    Because it was too crowded, we decided to move most of them under dropdown menu of the paragraph. 
    
    Shell we move this button into dropdown menu and assign keyboard short cut instead?


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---