Allow adding vanilla R plots to plot_grid about cowplot

[clauswilke]

Yes, this is a good idea. Can you prepare a pull request?

I think the way to go is to stay explicit, i.e., add a function that converts a base R plot into an object that plot_grid() can work with.

[flying-sheep]

so basically you just want me to add my gwrap_plot to the package (maybe rename it) and document it?

what name do you think would be fitting?

[clauswilke]

Yes, exactly. And maybe also mention in the plot_grid documentation that this exists.

Your name gwrap_plot sounds good to me. Alternatively, maybe plot_to_gtable?

[flying-sheep]

other alternatives: gtable_from_plot (analogous to your other gtable_* functions) or plot2gtable.

what do you prefer?

[clauswilke]

There's already ggplot_to_gtable in draw.R. I think this could go there as well, with a similar name (plot_to_gtable). Also, upon reconsideration, if you want to integrate this functionality into ggplot_to_gtable (so conversion is done automatically) that's fine by me as well.

[flying-sheep]

well, it’s not converting ggplots, so i could either

• leave two separate functions, or
• deprecate ggplot_to_gtable and make plot_to_gtable handle both ggplots and normal plots

[clauswilke]

I'm fine with either option, but have a weak preference for the latter.