fix!: refactor mutator icon

[BeksOmega]

The basics

• I branched from develop
• My pull request is against develop
• My code follows the style guide
• I ran npm run format and npm run lint

The details

Resolves

Fixes #7039

Proposed Changes

Refactors the mutator icon to conform to the IIcon interface.

Reason for Changes

Code consistency :D

Test Coverage

Updated tests! Also manual testing :D

Documentation

N/A

Additional Information

Dependent on #7110

Breaking changes / updating / upgrading

The Blockly.Mutator class has been renamed to Blockly.icons.MutatorIcon. If you use the blockly migration package, you can automatically rename this throughout your files by doing:

npx @blockly/migrate rename --from 9 --in-place ./path/to/my/files*

Note that you are now required to pass the block the mutator icon will live on to the mutator icon's constructor.

myBlock.addIcon(new Blockly.icons.MutatorIcon(['my_quark_block'], myBlock));

Misc methods

The updateBlockStyle method is no longer available. Updating block styles is handled automatically, so you should be able to safely remove any references to this method. If removing this causes problems for your application, please file a bug report.

The shouldIgnoreMutatorEvent method is no longer available. This method was originally intended for internal use, and should not have been public. If you were using this method, we recommend duplicating the code from here into your own application.

Deprecations

The Mutator.reconnect method has been deprecated. Use Connection.prototype.reconnect instead.

The findParentWs method has been deprecated. Use Workspace.prototype.getRootWorkspace instead.

Showing/hiding bubbles

The setVisible method has been renamed to setBubbleVisible for clarity. If you need to modify whether the bubble of the mutator icon is visible or not, you can do:

myBlock.getIcon(Blockly.icons.MutatorIcon.TYPE).setBubbleVisible(true);

The setVisible method will be removed in v11.

[cpcallen]

Not quite finished reviewing all of mutator_icon.ts, but thought I should send you my comments so far.

[cpcallen]

Not quite finished reviewing all of mutator_icon.ts, but thought I should send you my comments so far.

[cpcallen]

Finished reviewing; had a few additional comments.

[cpcallen]

Just a few small nits which I will leave for you to resolve as you see fit.