Skip to content

Conversation

lizlooney
Copy link
Collaborator

Fixes #248

In editor.ts:
Added code to onChangeAfterLoading to handle MOVE events. If the block being moved has a mrcOnMove method, call it.

In mrc_event.ts:
Added constants INPUT_TITLE, FIELD_PARAM_PREFIX, and WARNING_ID_NOT_IN_HOLDER. Removed call to ChangeFramework.registerCallback.
Removed onBlockChanged.
Added mrcHasWarning to EventMixin.
Added mrcOnLoad so that the block is checked when the block is loaded. Added mrcOnMove so that the block is checked when the block is moved.

Added code to onChangeAfterLoading to handle MOVE events. If the block being moved has a mrcOnMove method, call it.

In mrc_event.ts:
Added constants INPUT_TITLE, FIELD_PARAM_PREFIX, and WARNING_ID_NOT_IN_HOLDER.
Removed call to ChangeFramework.registerCallback.
Removed onBlockChanged.
Added mrcHasWarning to EventMixin.
Added mrcOnLoad so that the block is checked when the block is loaded.
Added mrcOnMove so that the block is checked when the block is moved.
@lizlooney lizlooney requested a review from alan412 September 29, 2025 06:41
Copy link
Collaborator

@alan412 alan412 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alan412 alan412 merged commit f525fa0 into wpilibsuite:main Sep 29, 2025
1 check passed
@lizlooney lizlooney deleted the pr_issue_248 branch October 2, 2025 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Event block warning should not become detached
2 participants