Plotly.js renderer

[rgbkrk]

This introduces a Plotly transform for the application/json+plotly.v1 mimetype.

/cc @jackparmer @chriddyp @captainsafia

[codecov-io]

Current coverage is 66.16% (diff: 52.63%)

Merging #662 into master will decrease coverage by 0.34%

@@             master       #662   diff @@
==========================================
  Files            28         30     +2   
  Lines           839        857    +18   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            558        567     +9   
- Misses          281        290     +9   
  Partials          0          0          

Powered by Codecov. Last update 7a609b8...5d3622e

[jdfreder]

slick

[chriddyp]

I'm not sure how the output cells work w.r.t. sizing, but we might need to set height here if layout.height && !layout.autosize. Here's an example figure:

data = [{x: [1, 2], y: [3, 1]}]
layout = {height: 1600}

[rgbkrk]

The cell allows the output to be any size it wants to be. We can set it explicitly here like you said.

render() {
  const { data, layout } = this.props.data.toJS();
  const style = {}
  if(layout.height && !layout.autosize) {
    style.height = layout.height;
  }
  return (
    <div style={style} ref={(el) => this.el = el} /> // eslint-disable-line
  )
}

[captainsafia]

Merged! 🎉

Slight coverage drop but we can address that in subsequent PRs.

[lock]

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.