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

[corneadoug]

Github user corneadoug commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/606#issuecomment-169854098
  
    I found one problem with this fix (or the new resize behavior).
    Let's take a look at the notebook **Zeppelin Tutorial**. At first it is 
showing some graphs.
    If we run a sql query (without the data), we will get an error which will 
burst out of the paragraph. (this PR fixes it).
    If we then run the whole notebook, the query will run successfully and show 
a graph, however that graph height will be the one of the previous error.
    <img width="521" alt="screen shot 2016-01-08 at 9 41 18 am" 
src="https://cloud.githubusercontent.com/assets/710411/12187077/01b32902-b5ec-11e5-811f-8e885ef6dd61.png";>
    
    Overall, the resizing aspect of the output is a bit tricky between content 
based height and manually set height.
    TEXT type of output as well as HTML and ANGULAR are content based height.
    Only TABLE type of output is responding to manually set height.


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