[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-08-31 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1241 Merging if there is 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

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-08-31 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1241 @AhyoungRyu Just pointing out that using a Table component to show the config variable isn't great. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-08-30 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1241 @corneadoug I'm not sure I got your point correctly, "You can configure Apache Zeppelin with both environment variables in `conf/zeppelin-env.sh` (`conf\zeppelin-env.cmd` for Windows) and

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-08-30 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1241 LGTM, could be good to try changing that Table by something more descriptive. Although the fact that there is 2 different configuration files with possible exclusive configurations doesn't

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-08-25 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1241 LGTM :smile: --- 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

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-08-25 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1241 Is this good to go now? --- 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

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-31 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1241 @zjffdu Yeah I also struggled with ruby and my OSX version issues. Anyway thanks for providing the link :) --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-31 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1241 Thanks @AhyoungRyu , the new UI lgtm. BTW, this link helps me in case someone else need this, http://pragmaticstudio.com/blog/2010/9/23/install-rails-ruby-mac --- If your project is set up

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1241 @zjffdu I'm not sure, your issue is related with [this issue](https://github.com/bundler/bundler/issues/4065#issuecomment-149596385), could you try ``` gem uninstall bundler

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-29 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1241 yes, I run it under `ZEPPELIN_HOME/docs` --- 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

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1241 @zjffdu Did you run `bundle install` under `ZEPPELIN_HOME/docs` dir? --- 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] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-29 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1241 @AhyoungRyu I follow the instruction here to build the doc https://github.com/apache/zeppelin/blob/master/docs/README.md#build-documentation, but fails. Here's the error after `bundle install`

[GitHub] zeppelin issue #1241: [ZEPPELIN-1217] Remove horizontal scrollbar in Zeppeli...

2016-07-29 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1241 @zjffdu Could you review this PR as a reporter? --- 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