[GitHub] incubator-zeppelin pull request: Change the placement of 'Reposito...

2016-05-19 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/901#issuecomment-220338513 @AhyoungRyu If tooltip display is the only issue you can try other placement properties `tooltip-placement="left"` otherw

[GitHub] incubator-zeppelin pull request: Update HttpClient Dependencies

2016-05-18 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/897#issuecomment-220091243 LGTM --- 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

[GitHub] incubator-zeppelin pull request: Fixing Zeppelin-838: Minor improv...

2016-05-18 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/872#issuecomment-220091216 LGTM --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-859 Error on notebook pa...

2016-05-18 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/896 ZEPPELIN-859 Error on notebook page due null property ### What is this PR for? Add null check to prevent js error on notebook page ### What type of PR is it? Bug Fix

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-05-12 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-218786264 @corneadoug I'm following up on handsontable repo. Will find some time to explore for a potential PR. --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-05-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-218190257 @corneadoug Again, this is an issue on handsontable please follow these links. Those entries in index.html should disappear when handsontable team fix

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-05-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-218142389 Not sure if we can avoid those dependencies by choosing hot-builder! Though we are not using the copy/ paste, date picker and date formatting

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-05-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-218138415 @corneadoug [Use without pikaday #2841](https://github.com/handsontable/handsontable/issues/2841) --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-05-09 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-218056874 @prabhjyotsingh conflict resolved f13ba91 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-05-05 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-217091158 @felixcheung this PR is only for improving table performance. CSV/TSV export is in #725 and #714 --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-05-04 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-216953422 @Leemoonsoo the only pending item is the sort header arrow style. Should we try something more stylish ? https://cloud.githubusercontent.com/assets

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-05-02 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-216121658 @Leemoonsoo license doc updated and rebased 2e422ea 4fbb22b --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-04-28 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-215517988 @Leemoonsoo that's in the todo list. Can be done either in FE or in server. Thoughts? --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-04-27 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-215308849 Right, it is supported only in pro ver. Handsontable's Export feature is in front-end. IMHO export should be a backend feature and will be a good addition

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-04-27 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-215040456 Won't affect #714 and #725 #6 and #761 are using jQuery Datatables This PR is a proposal to use Handsontable instead of datatables

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-04-26 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/858 ZEPPELIN-830 Improve table display to handle large data ### What is this PR for? This is an improvement for table display. By using [Handsontable](https://github.com/handsontable

[GitHub] incubator-zeppelin pull request: Add tooltip for notes reload butt...

2016-04-05 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/813#issuecomment-205899745 LGTM :+1: --- 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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-672] Add feature to exp...

2016-04-05 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-205768826 > #761 to replace the table only (if it brings better performances) What is the max number of rows this lib can support ? --- If your proj

[GitHub] incubator-zeppelin pull request: ZEPPELIN-580 Add support for mult...

2016-03-30 Thread r-kamath
Github user r-kamath closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/805 --- 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

[GitHub] incubator-zeppelin pull request: [WIP] ZEPPELIN-580 Add support fo...

2016-03-29 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/805 [WIP] ZEPPELIN-580 Add support for multi select in form select box ### What is this PR for? This feature allows the user to choose single/multiple options in a form select box

[GitHub] incubator-zeppelin pull request: replace standard alert and confir...

2016-03-29 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/802#issuecomment-202828599 LGTM --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-748 make websocket maxTe...

2016-03-19 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/784 ZEPPELIN-748 make websocket maxTextMessageSize configurable ### What is this PR for? Allow the user to modify the value for websocket maxTextMessageSize via conf. Recently a user

[GitHub] incubator-zeppelin pull request: Hot fix grunt failure

2016-03-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/772#issuecomment-194766282 looks like the failure is not related to this change 31m*** 1 TEST FAILED *** [INFO

[GitHub] incubator-zeppelin pull request: Hot fix grunt failure

2016-03-09 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/772 Hot fix grunt failure ### What is this PR for? This is a fix for grunt failure due to jshint warnings Grunt error: `src/app/notebook/paragraph/paragraph.controller.js

[GitHub] incubator-zeppelin pull request: Fix Selenium test url

2016-03-09 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/771 Fix Selenium test url ### What is this PR for? use env variable `url` instead of system property ### What type of PR is it? Bug Fix ### What is the Jira issue

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-698] Change shortcut fo...

2016-03-02 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/756#issuecomment-191355294 nice improvement. LGTM --- 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

[GitHub] incubator-zeppelin pull request: Add filter to quickly search the ...

2016-02-07 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/660#issuecomment-181240326 @minahlee I've updated the css. Also removed the custom clear button for the search box which wasn't working well with this css update. --- If your

[GitHub] incubator-zeppelin pull request: Propagate maven -DskipTests optio...

2016-02-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/682#issuecomment-179099186 :+1: --- 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

[GitHub] incubator-zeppelin pull request: Add filter to quickly search the ...

2016-02-02 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/660#issuecomment-179029777 @corneadoug padding fixed in ed41d41 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-zeppelin pull request: Add filter to quickly search the ...

2016-01-27 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/660#issuecomment-175597946 @corneadoug thanks for the comments. please take a look at updates --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: Fix build failure due to latest b...

2016-01-26 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/677 Fix build failure due to latest bower (1.7.5) ### What is this PR for? Build fails due to new bower version 1.7.5. This PR tries to fix this issue by adding the previous version

[GitHub] incubator-zeppelin pull request: [Hot Fix] Fix build failure due t...

2016-01-26 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/677#issuecomment-175465448 Travis build failure is not related to this change https://cloud.githubusercontent.com/assets/2031306/12606910/1cf5d024-c4f7-11e5-99d9-0bcc6d3569f7.png

[GitHub] incubator-zeppelin pull request: Add filter to quickly search the ...

2016-01-20 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/660 Add filter to quickly search the notebook from the list ### What is this PR for? Allows the user to instantly find the notebook from the list shown on the home page

[GitHub] incubator-zeppelin pull request: ZEPPELIN-593 Test Paragraph actio...

2016-01-17 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/619#issuecomment-172448650 @bzz @Leemoonsoo ready for review --- 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

[GitHub] incubator-zeppelin pull request: Update default notebook generateN...

2016-01-15 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/633#issuecomment-171914279 @felixcheung @prabhjyotsingh travis fails for `SPARK_VER="1.4.1" HADOOP_VER="2.3" PROFILE="-Pspark-1.

[GitHub] incubator-zeppelin pull request: ZEPPELIN-593 Test Paragraph actio...

2016-01-13 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/619#issuecomment-171232018 @bzz I've updated the patch as per the review comment. Thanks --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-zeppelin pull request: Update default notebook generateN...

2016-01-13 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/633 Update default notebook generateName function for create note dialog ### What is this PR for? The current default name for a notebook in create dialog is random, which is difficult

[GitHub] incubator-zeppelin pull request: ZEPPELIN-593 Test Paragraph actio...

2016-01-10 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/619 ZEPPELIN-593 Test Paragraph action Disable run and refactor common methods ### What is this PR for? Add new selenium test for disable paragraph action and refactor some

[GitHub] incubator-zeppelin pull request: ZEPPELIN-565 ZEPPELIN-564 Paragra...

2016-01-06 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/606 ZEPPELIN-565 ZEPPELIN-564 Paragraph UI bugs ### What is this PR for? Bug fix paragraph UI ### What type of PR is it? Bug Fix ### Is there a relevant Jira issue

[GitHub] incubator-zeppelin pull request: update error message and output s...

2016-01-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/583#issuecomment-168601911 @Leemoonsoo @corneadoug cool. Thanks for the feedback --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-zeppelin pull request: update error message and output s...

2016-01-03 Thread r-kamath
Github user r-kamath closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/583 --- 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

[GitHub] incubator-zeppelin pull request: update error message and output s...

2015-12-30 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/583 update error message and output style ### What is this PR for? Styling text and error output ### What type of PR is it? Improvement ### How should

[GitHub] incubator-zeppelin pull request: ZEPPELIN-523 ] table scrolling bu...

2015-12-20 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/556#issuecomment-166117118 @cloverhearts Thanks for the fix. LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-zeppelin pull request: Add firebug and firepath for debu...

2015-12-17 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/490#issuecomment-165386214 @prabhjyotsingh please rebase --- 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

[GitHub] incubator-zeppelin pull request: Option to restart interpreter fro...

2015-12-17 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/480#issuecomment-165386267 @prabhjyotsingh rebase --- 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

[GitHub] incubator-zeppelin pull request: Fix typo / grammar from rest-note...

2015-12-17 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/545#issuecomment-165421322 LGTM :+1: --- 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

[GitHub] incubator-zeppelin pull request: fixing z-index for all modals, ba...

2015-12-15 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/535#issuecomment-164823355 :+1: LGTM --- 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

[GitHub] incubator-zeppelin pull request: Interpreter properties editor upd...

2015-12-15 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/525#issuecomment-165015122 @corneadoug border-top looks good --- 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

[GitHub] incubator-zeppelin pull request: Replace standard alert and confir...

2015-12-14 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/501#issuecomment-164659614 > Neither the cancel nor the OK button has focus on them. Would be nice to have either one pre-focussed. @gauravkumar37 thanks for the feedback. I

[GitHub] incubator-zeppelin pull request: Interpreter properties editor upd...

2015-12-14 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/525#issuecomment-164673854 @corneadoug thanks for the review. Removed the table header style and other changes in de75e49 --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: [WIP] fix bootstrap dialog defaul...

2015-12-14 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/532 [WIP] fix bootstrap dialog default button ### What is this PR for? Fix default focus issue on bootstrap alert/confirm box ### What type of PR is it? Improvement

[GitHub] incubator-zeppelin pull request: Interpreter properties editor upd...

2015-12-09 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/525 Interpreter properties editor updates - Add extra validations in add new interpreter property function to prevent 'undefined' - Instead of an empty table show a message when

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

2015-12-09 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163164018 :+1: for bookmarklet --- 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

[GitHub] incubator-zeppelin pull request: Add/pr teamplate

2015-12-09 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/523#issuecomment-163163806 @corneadoug can you also add PR status in readme ? `[![Issue Stats](http://issuestats.com/github/apache/incubator-zeppelin/badge/pr)](http

[GitHub] incubator-zeppelin pull request: Edit title fix for Notebook and P...

2015-12-09 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/512#issuecomment-163179592 @corneadoug try merging master on #324 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-zeppelin pull request: Add more unit tests for notebook ...

2015-12-07 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/470#issuecomment-162451893 re-defining`console.log = function() {}` in main.js works! We can use this till we get `captureConsole: false` working. Thank you --- If your project

[GitHub] incubator-zeppelin pull request: Add more unit tests for notebook ...

2015-12-07 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/470#issuecomment-162448468 @corneadoug `captureConsole` isn't working for me too. Two options I can think of are, 1) Let the tests print the log messages on console 2) Remove

[GitHub] incubator-zeppelin pull request: Edit title fix for Notebook and P...

2015-12-07 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/512#issuecomment-162518422 @corneadoug fixed --- 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

[GitHub] incubator-zeppelin pull request: Replace standard alert and confir...

2015-12-05 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/501#issuecomment-162218643 @corneadoug we can remove the title box if necessary. Even in the standard alert/confirm there is nothing in the title box. IMHO the standard ones

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-484] A small utility to...

2015-12-04 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/513#issuecomment-162049203 @Leemoonsoo I do `curl https://patch-diff.githubusercontent.com/raw/apache/incubator-zeppelin/pull/<PR#>.patch | git apply` in master, review it and

[GitHub] incubator-zeppelin pull request: Edit title fix for Notebook and P...

2015-12-04 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/512#issuecomment-162040934 @prabhjyotsingh adding transition requires too many other changes. Planning to do it in a different ticket/PR --- If your project is set up for it, you

[GitHub] incubator-zeppelin pull request: Fix navbar-brand image margin

2015-12-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/500#issuecomment-161577146 :+1: --- 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

[GitHub] incubator-zeppelin pull request: Replace standard alert and confir...

2015-12-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/501#issuecomment-161613487 I was waiting for CI to pass. Ready for review --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: Cron UI fix and css cleanup

2015-12-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/511#issuecomment-161568503 @corneadoug 87c21cd https://cloud.githubusercontent.com/assets/2031306/11556938/ce510db4-99cf-11e5-8f55-e15704f9e4cb.png;> --- If your project is

[GitHub] incubator-zeppelin pull request: Cron UI fix and css cleanup

2015-12-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/511#issuecomment-161568060 @corneadoug currently it is 2px. will update to 5px --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: Don't set Dirty state if text is ...

2015-12-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/505#issuecomment-161570344 :+1: --- 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

[GitHub] incubator-zeppelin pull request: Add animation for dropdown menu

2015-12-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/479#issuecomment-161573209 @corneadoug my idea was to have some animation instead of `display:block`. 0.2s sounds good to me. 55f9969 --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: Cron UI fix and css cleanup

2015-12-03 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/511 Cron UI fix and css cleanup - add spacing between links in presets - remove inline css and fix alignment Before https://cloud.githubusercontent.com/assets/2031306

[GitHub] incubator-zeppelin pull request: ZEPPELIN-480 Enter/return key on ...

2015-12-03 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/509#issuecomment-161571362 LGTM! --- 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

[GitHub] incubator-zeppelin pull request: Edit title fix for Notebook and P...

2015-12-03 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/512 Edit title fix for Notebook and Paragraph - enable autofocus to input field *(no need to click twice to edit title)* - fix esc key handler on edit input to revert the change *(edit

[GitHub] incubator-zeppelin pull request: [WIP] Replace standard alert and ...

2015-12-01 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/501 [WIP] Replace standard alert and confirm with BootstrapDialog Before https://cloud.githubusercontent.com/assets/2031306/11523424/303e835a-98e8-11e5-9f97-c4618b2905ee.png

[GitHub] incubator-zeppelin pull request: ZEPPELIN-462 - Fix Navbar CSS

2015-11-27 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/483#issuecomment-160080960 tested. LGTM :+1: --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-467 Fix ace editor focus...

2015-11-27 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/482#issuecomment-160082731 :+1: LGTM. works well on Chrome and Firefox --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-zeppelin pull request: Fix paragraph actions dropdown st...

2015-11-26 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/478 Fix paragraph actions dropdown styling for Firefox Before on Firefox https://cloud.githubusercontent.com/assets/2031306/11418764/ff5f1bc6-9448-11e5-9eb6-cf0f6f79be2e.png

[GitHub] incubator-zeppelin pull request: Fix 'create' -> 'cancel' -> 'crea...

2015-11-26 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/473#issuecomment-159868995 tested on local. works fine. LGTM! --- 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

[GitHub] incubator-zeppelin pull request: [Minor] Possible to make a hang w...

2015-11-25 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/466#issuecomment-159811505 Looks good to me --- 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

[GitHub] incubator-zeppelin pull request: Cleanup redundant css units and u...

2015-11-25 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/476 Cleanup redundant css units and use shorthand property for background You can merge this pull request into a Git repository by running: $ git pull https://github.com/r-kamath

[GitHub] incubator-zeppelin pull request: ZEPPELIN-463: UI freezes if table...

2015-11-25 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/472#issuecomment-159612911 @prabhjyotsingh is there a way to tweak Perfect Scrollbar to make it work on large dataset ? --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-461] fix typos in geode...

2015-11-25 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/471#issuecomment-159811414 LGTM :+1: --- 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

[GitHub] incubator-zeppelin pull request: Add more unit tests for notebook ...

2015-11-24 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/470 Add more unit tests for notebook controller You can merge this pull request into a Git repository by running: $ git pull https://github.com/r-kamath/incubator-zeppelin

[GitHub] incubator-zeppelin pull request: Add basic documentation for Markd...

2015-11-24 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/467#issuecomment-159509640 @felixcheung thanks for the suggestion. I've included the markdown4j link in d839992 --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: ZEPPELIN-413: Fix ability to link...

2015-11-23 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/464#issuecomment-159150182 :+1: LGTM --- 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

[GitHub] incubator-zeppelin pull request: Add basic documentation for Markd...

2015-11-23 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/467 Add basic documentation for Markdown You can merge this pull request into a Git repository by running: $ git pull https://github.com/r-kamath/incubator-zeppelin markdownDocUpdate

[GitHub] incubator-zeppelin pull request: ZEPPELIN-387 Allow enable/disable...

2015-11-20 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/414#issuecomment-158579034 @corneadoug @Leemoonsoo build issues are fixed --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: ZEPPELIN-447: removing of redunda...

2015-11-19 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/449#issuecomment-158306521 :+1: --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-387 Allow enable/disable...

2015-11-17 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/414#issuecomment-157418759 @corneadoug fixed the z.run() case in bdf703d --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: ZEPPELIN-426 - Allowing new parag...

2015-11-17 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/440#issuecomment-157420527 LGTM! --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-423 Exclude DEPENDENCIES...

2015-11-17 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/434#issuecomment-157451614 LGTM --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-418 Keep paragraph heigh...

2015-11-17 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/426#issuecomment-157452053 LGTM --- 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

[GitHub] incubator-zeppelin pull request: [WIP] ZEPPELIN-52 Long statements...

2015-11-17 Thread r-kamath
Github user r-kamath closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/415 --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-387 Allow enable/disable...

2015-11-15 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/414#issuecomment-156930130 @Leemoonsoo @corneadoug @jongyoul updates from 3a7175a https://cloud.githubusercontent.com/assets/2031306/11175456/6b5e88dc-8c58-11e5-99ff

[GitHub] incubator-zeppelin pull request: [WIP] ZEPPELIN-52 Long statements...

2015-11-11 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155704705 >what do you think make it aware contents length of 1st line and hide (or make it almost transparent) control only when there're text underneath, and s

[GitHub] incubator-zeppelin pull request: [WIP] ZEPPELIN-52 Long statements...

2015-11-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155457111 @Leemoonsoo how about we hide the control bar and show it on mouseover event of the paragraph --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: ZEPPELIN-387 Allow enable/disable...

2015-11-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/414#issuecomment-155385289 Hi @corneadoug, That sounds like a right spot for the enable/disable menu item. Updated in 87aa4e8. Please take a look at the new screenshots. Thanks

[GitHub] incubator-zeppelin pull request: ZEPPELIN-52 Long statements in pa...

2015-11-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155382830 @prabhjyotsingh tested on Firefox and IE :tada: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: ZEPPELIN-52 Long statements in pa...

2015-11-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155420038 @Leemoonsoo Thanks for the document. >...without adding more empty spaces inside of Paragraph... makes sense. working on a better solut

[GitHub] incubator-zeppelin pull request: ZEPPELIN-52 Long statements in pa...

2015-11-09 Thread r-kamath
GitHub user r-kamath reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/415 ZEPPELIN-52 Long statements in paragraph and going beneath the control bar before https://issues.apache.org/jira/secure/attachment/12726406/Screen%20Shot%202015-04-18%20at

[GitHub] incubator-zeppelin pull request: ZEPPELIN-52 Long statements in pa...

2015-11-09 Thread r-kamath
Github user r-kamath closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/415 --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-52 Long statements in pa...

2015-11-09 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/415 ZEPPELIN-52 Long statements in paragraph and going beneath the control bar before https://issues.apache.org/jira/secure/attachment/12726406/Screen%20Shot%202015-04-18%20at

[GitHub] incubator-zeppelin pull request: ZEPPELIN-387 Allow enable/disable...

2015-11-09 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/414 ZEPPELIN-387 Allow enable/disable of specific paragraphs - When "run all" is called the disabled paragraphs are not executed. - There is no "run" button for

  1   2   >