Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ZEPPELIN-1804] Added Link this paragraph to keyboard shortcut #1756

Closed
wants to merge 1 commit into from

Conversation

soralee
Copy link
Contributor

@soralee soralee commented Dec 14, 2016

What is this PR for?

Each paragraph menu, it has "Link this paragraph" function which is link each of the paragraph but there is not shortcut key.

What type of PR is it?

[ Improvement ]

Todos

  • None

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-1804

How should this be tested?

Please press Ctrl+Alt+w shortcut key and check work as expected well

Screenshots (if appropriate)

image

z1804_1

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@soralee soralee closed this Dec 14, 2016
@soralee soralee reopened this Dec 14, 2016
@soralee soralee closed this Dec 14, 2016
@soralee soralee reopened this Dec 14, 2016
@soralee soralee closed this Dec 14, 2016
@soralee soralee reopened this Dec 14, 2016
@soralee soralee closed this Dec 14, 2016
@soralee soralee reopened this Dec 14, 2016
@soralee soralee changed the title [ZEPPELIN-1804] Added "Link this paragraph" to keyboard shortcut [ZEPPELIN-1804] Added Link this paragraph to keyboard shortcut Dec 14, 2016
@soralee soralee closed this Dec 14, 2016
@soralee soralee reopened this Dec 14, 2016
@soralee soralee force-pushed the ZEPPELIN-1804 branch 2 times, most recently from 48b225b to 750f5e4 Compare December 14, 2016 12:10
@tae-jun
Copy link
Contributor

tae-jun commented Dec 14, 2016

OMG hope that it passes this time... :)

@soralee
Copy link
Contributor Author

soralee commented Dec 14, 2016

@tae-jun Sorry for many reopened PR.
I had a weird CI problem like the link (https://travis-ci.org/apache/zeppelin/builds/183881539). But I solved with recommit PR. Now, CI fails again 😿

@Leemoonsoo
Copy link
Member

Tested and LGTM.

@soralee soralee force-pushed the ZEPPELIN-1804 branch 4 times, most recently from 3863216 to b6f5453 Compare December 15, 2016 05:06
@soralee
Copy link
Contributor Author

soralee commented Dec 15, 2016

Finally, the CI is GREEN! (After about 7 attempts) 😭
It's time to ready to review.

@AhyoungRyu
Copy link
Contributor

At last! Tested and it works well. LGTM 👍

@soralee
Copy link
Contributor Author

soralee commented Dec 15, 2016

@Leemoonsoo @AhyoungRyu Thanks for review :)

@Leemoonsoo
Copy link
Member

Merge to master if there're no more comments.

@asfgit asfgit closed this in 14479fd Dec 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants