fix: respect the top-level copyButtons option

[rafegoldberg]

🎫 Resolves RM-10635	🐙 Closes #964

🧰 Changes

The main RMDX.compile() method should respect the copyButtons param when rendering our code block components.

• Accept the copyButtons option—we used to set this param directly on our custom Unified processor, but since Remark actually freeze's it's processor instance we can achieve a similar effect by passing it directly to the CodeTabsTransformer plugin as a configuration option:
  

  markdown/lib/compile.ts

  Line 28 in 96f9644

  [codeTabsTransformer, { copyButtons }],

• Add a Copy Buttons toggle to the dev playground.¹

• Miscellaneous detailing and minor cleanup.

🧬 QA & Testing

Pull this PR down, run the start script, and navigate to the “Code Blocks” example. Try hovering over a code block to verify that the copy button is only shown if/when the Copy Buttons toggle is checked.

Footnotes

1. @kellyjosephprice—do we even respect those other two global config options? I think safeMode and lazyImages both have toggles in the playground, but there are a bunch of others which I'm not sure are still valid either? Anyways, just an aside, but it would be nice to clean this all up at some point! ↩

[kellyjosephprice]

1. @kellyjosephprice—do we even respect those other two global config options? I think safeMode and lazyImages both have toggles in the playground, but there are a bunch of others which I'm not sure are still valid either? Anyways, just an aside, but it would be nice to clean this all up at some point! ↩

I think the still valid options are:

  copyButtons: true,
  markdownOptions: {
    fences: true,
    commonmark: true,
    gfm: true,
    ruleSpaces: false,
    listItemIndent: '1',
    spacedTable: true,
    paddedTable: true,
  },
  lazyImages: true,,
  theme: 'light',

I don't think we're passing markdownOptions in correctly, or if they're they're up to date with the latest options.

[kellyjosephprice]

LGTM.

Just now seeing that Settings is not considered extensible.

Another alternative would be to pass this down through a context. This is great though.

[trishaprile]

🙏

[rafegoldberg]

(gonna hold of on cutting a new release for this pending #992 so we can hopefully kill two birds w/one stone)

[rafegoldberg]

This PR was released!

🚀 Changes included in v7.6.7