Results 31 comments of Matt Mower

> @whawker - Is this an acceptable approach? Or do you want to do a more complete package update? Or maybe @ocavue could weigh in?

> Would you be interested in reworking this extension as as component instead, and opening a PR against the `v3` branch? @whawker - thanks for taking a look! Most of...

Is manual positioning sufficient for the case where both language-select and format are desired? ```tsx ``` Or should they auto position relative to each other when both are included?

> Good point, there should probably be a parent "menu" component that is responsible for positioning with the code block, then you pass the functional buttons as children. @whawker -...

@ocavue - I know the prettier v3 update was [attempted before](https://github.com/remirror/remirror/pull/2139) and then [reverted](https://github.com/remirror/remirror/pull/2149), but I've taken a different approach here (async formatting). Would you be willing to review? In...

@whawker @ocavue - I've rebased this PR and the code formatter ~~extension~~ component PR #2256 on branch `v3`. ~~It looks like the `pr-name-linter` CI check is failing, but that seems...

@whawker - Thanks for reviewing. I believe all feedback has been addressed in [20479ec](https://github.com/remirror/remirror/pull/2255/commits/20479ec7e15788999624825fea6615a2eba6d28a). Happy to revise more if you see anything else. Could you help with the included changeset?...

> Thanks for highlighting the issue on the changeset, if we update the code block extension itself (and the `remirror` entry point) it should suffice. Thanks for cleaning this up....

@bwiernik - Would you be willing to review this?