mpld3/mplexporter — pull request #7 by theengineear (2014-03-03 05:26:31+00:00)
Title: Plolty's got: docstrings, updated draw_text for 'text_type' option, fixed conflicts

I'm pretty sure this should merge without conflict. I added a ton of documentation and updated the plotly renderer to work with the text_type option the exporter passes in the draw_text call. Just a few notes in no particular order of importance.

- All titles are getting put into plotly as annotations right now, there isn't a better way to do this when you have multiple plot titles for subplots.
- I really like having the renderer draw xlabels, ylabels, title's as a general draw_text call. I really think it improves the code.
- You might consider changing the get_axes_properties function to access x and y axes by a key instead of a list:

```
props = {'axesbg': color_to_hex(ax.patch.get_facecolor()),
'axesbgalpha': ax.patch.get_alpha(),
'bounds': ax.get_position().bounds,
'dynamic': ax.get_navigate(),
'axes': [get_axis_properties(ax.xaxis),
get_axis_properties(ax.yaxis)]}
```

When you need to use this in the renderer from the axes props, the call looks like this when you want to see if an axis has the grid on:

```
props['axes'][0]['grid']['gridOn']
```

I think it might make more sense to do this:

```
props['axes']['x']['grid']['gridOn']
```

Keep up the awesome progress! I'm enjoying the scramble to keep things working!