Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.1.12: Support for backpack plugin integration & Fix several bugs #38

Merged
merged 8 commits into from
Feb 27, 2024

Conversation

HollowMan6
Copy link
Collaborator

@HollowMan6 HollowMan6 commented Jan 16, 2024

  • Fix the context menu error when integrating with backpack plugin
    Right-click on a block in the backpack causes the following errors:
Cannot read properties of undefined (reading 'size') TypeError: Cannot read properties of undefined (reading 'size')
    at Object.preconditionFn (webpack-internal:///./src/multiselect_contextmenu.js:67:27)
......

@HollowMan6
Copy link
Collaborator Author

WX20240116-201546@2x

@HollowMan6 HollowMan6 marked this pull request as draft January 16, 2024 18:25
@HollowMan6 HollowMan6 marked this pull request as ready for review January 16, 2024 18:31
@HollowMan6 HollowMan6 changed the title Fix the context menu error when integrating with backpack plugin v0.1.12: Support for backpack plugin integration & Fix several bugs Jan 16, 2024
@HollowMan6 HollowMan6 force-pushed the main branch 2 times, most recently from a9e78d5 to 9a73da6 Compare January 17, 2024 13:39
@HollowMan6 HollowMan6 requested a review from ewpatton January 21, 2024 16:07
@HollowMan6 HollowMan6 marked this pull request as draft February 8, 2024 11:07
@HollowMan6
Copy link
Collaborator Author

Let's wait for the adaptation to Blockly 11 to be completed (#39) before we land this PR.

@ewpatton
Copy link
Member

ewpatton commented Feb 8, 2024

@HollowMan6 Sure thing.

@HollowMan6 HollowMan6 marked this pull request as ready for review February 20, 2024 14:35
@HollowMan6
Copy link
Collaborator Author

@ewpatton Let's merge this and release a new version (v0.1.12). This can be our last stable version for Blockly v10.

I suddenly became aware that we will have significant breaking changes for Blockly v11 (#39), so I guess it will be better to raise the minor version number to v0.2.x or the major version number to v1.0.x since I can't guarantee that it will still be as stable as the old ones.

> Let's wait for the adaptation to Blockly 11 to be completed (#39) before we land this PR.

I will then open a separate PR for this.

package.json Show resolved Hide resolved
Cannot read properties of undefined (reading 'size')
TypeError: Cannot read properties of undefined (reading 'size')
    at Object.preconditionFn (webpack-internal:///./src/multiselect_contextmenu.js:67:27)

Signed-off-by: Songlin Jiang <[email protected]>
Signed-off-by: Songlin Jiang <[email protected]>
Signed-off-by: Songlin Jiang <[email protected]>
@HollowMan6 HollowMan6 marked this pull request as ready for review February 27, 2024 18:24
Require blockly version to be 10.2.0-11 (as of mit-cml#39)

Signed-off-by: Songlin Jiang <[email protected]>
…kpack

We must listen events of all workspaces, not only the binded one, to cover
all possible selection changes

Signed-off-by: Songlin Jiang <[email protected]>
@laurensvalk
Copy link
Contributor

Thanks for the fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants