-
Notifications
You must be signed in to change notification settings - Fork 1
Migrate to modern Blockly #75
Copy link
Copy link
Labels
blockThings related to clipcc-block.Things related to clipcc-block.guiThings related to clipcc-gui.Things related to clipcc-gui.help wantedExtra attention is neededExtra attention is neededvmThings related to clipcc-vm.Things related to clipcc-vm.
Milestone
Metadata
Metadata
Assignees
Labels
blockThings related to clipcc-block.Things related to clipcc-block.guiThings related to clipcc-gui.Things related to clipcc-gui.help wantedExtra attention is neededExtra attention is neededvmThings related to clipcc-vm.Things related to clipcc-vm.
Type
Fields
Give feedbackNo fields configured for Task.
Description
Since the Scratch foundation has redone scratch-blocks based on the latest version of Blockly, the scratch-blocks used by clipcc are outdated and need to be upgraded
urgently.Proposed Changes
Related upstream branch
https://github.com/scratchfoundation/scratch-gui/tree/spork
https://github.com/scratchfoundation/scratch-vm/tree/spork
https://github.com/scratchfoundation/scratch-blocks/tree/spork
https://github.com/scratchfoundation/scratch-editor/tree/spork
Related Pull Requests
Todo Lists
Core Features
✨ feat(block): Scratch-styled comments #90
Blocks
Insertion marker should shown as Scratch #88
Toolbox
Solution: Override
setMetrics_to update selected category.Fields
Blockly.getMainWorkspacein initView (in FieldMatrix) since mainWorkspace hasn't set when initializing the toolbox. (scratch-blocks spork may also meet the issue)Solution: Add
FieldMatrix.getWorkspaceOptionto get correct workspace.matrixhas the color of pen, it is not a good solution.Solution: Override
applyColourto update source block's color.Solution: Modify
FieldMatrix.doClassValidation_.Solution: Fix condition in
FieldMatrix.doValueUpdate_.notewhile the block definition usesvalueinblocks_common/note.js. This also occurs in scratch-blocks.Solution: Replace
valuewithnotein block definition.FieldNumberdoesn't allow empty value('') and doesn't show number pad in touch devices. We might have to implement another Field class. ✨ feat(block): implement ScratchNumberField #110Solution: Override
zelos.ConstantProvider.setFontConstants_to prevent update ofFIELD_DROPDOWN_BORDER_RECT_HEIGHT. It should be fixed to 8 * GRID_UNIT🐛 fix(block): prevent update of
FIELD_DROPDOWN_BORDER_RECT_HEIGHT#89domToFieldVariableand its behaviors.Developer Enhancements
.eslintrctoeslint.config.mjs, to support typescript-eslint. ReferenceCompatibility Issues
Integration
Other Plans
Field.updateSize_should deal withmarginargument.Field.SKIP_SETUP. ✨ feat(block): implement ScratchNumberField #110Warnings Related to Blockly