Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1991
Some CI tests are failed but it's not related to this change. I'm merging
now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1991
@1ambda Looks good! Will merge after CI tests are done if there are no
comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1991
@felixcheung Fixed. (`font-size: 12`)
---
If your project is set up for
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1991
@1ambda CI is failed only in Selenium test like below
```
Caused by: org.openqa.selenium.NoSuchElementException: Unable to locate
element:
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1991
Merge into master and branch-0.7 if there are no comments
---
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 user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1991
@AhyoungRyu thanks for review. CI failed but irrelavent
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1991
Thanks for taking care this @1ambda! Tested and looks nice.
---
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
