close
Skip to content

onRender should support more than a bare function #326

Description

@cpsievert

I'd expect this to work, but it currently doesn't:

library(plotly)
library(reactR)

plot_ly() %>%
    onRender(babel_transform("gd => console.log(gd)"))

That's because babel_transform() doesn't return a bare function:

cat(babel_transform("gd => console.log(gd)"))
"use strict";

(function (gd) {
  return console.log(gd);
});

However, HTMLWidgets.evalAndRun assumes we're passing a bare function:

var taskFunc = eval("(" + task + ")");

Seems as though we could support this use case by also trying var taskFunc = eval(task);

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions