search

observablehq / feedback

Customer submitted bugs and feature requests
42 stars 3 forks source link

"eject to JavaScript" option in the cell header of chart cell missing #559

Closed urswilke closed 1 year ago

urswilke commented 1 year ago

Sorry, if I'm missing something obvious, but it seems as if the "eject to JavaScript option () in the cell header" (As described here (in the last paragraph)) has disappeared. Quite sure it was there yesterday..

Desktop:

mootari commented 1 year ago

Apologies for the confusion! We moved the option to the "add cell" menu below the chart cell:

image

I'll leave this issue open until we've updated our docs.

tophtucker commented 1 year ago

All my bad! I've updated the chart cell docs. (Data table docs will be updated Monday.) I'm sorry for this; I know it feels like a sort of cruel joke to move something right after launching it… a couple other things might move yet, but we'll make sure to keep the docs in sync.

Always tough to balance getting things out when we know there's more to be done, vs. refining in private so we don't move things around on you! In this case we erred on the side of the former; the upside is that getting that outside perspective is invaluable to help us prioritize everything we could be doing.

While I've got you here: currently, "Convert to JavaScript" converts to a Plot.auto mark, which itself creates a dot, rule, bar, rect, etc. We have a notebook that converts Plot.auto to those explicit marks. Would it be valuable to you to have the option to convert the chart cell to that sort of explicit mark instead?

urswilke commented 1 year ago

No worries!

Yes, I think converting to explicit marks would be valuable.

tophtucker commented 1 year ago

Thanks @urswilke. I'm going to close this issue since the original documentation issue has been resolved; I split off the explicit marks thing to https://github.com/observablehq/feedback/issues/560. Feel free to open a new issue for anything else I haven't covered.