Skip to content

Fix missing error messages on variable fields - #9078

Merged
4ian merged 5 commits into
masterfrom
fix-variable-field-error
Sep 9, 2026
Merged

Fix missing error messages on variable fields#9078
4ian merged 5 commits into
masterfrom
fix-variable-field-error

Conversation

@D8H

@D8H D8H commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • Errors are displayed for any undeclared child variables like they already are in the event sheet.
  • Error inside brackets are now displayed (but it still doesn't display the error location).

Regressions

  • This error is no longer displayed but I think it's ok because the editor doesn't allow to declare variables with space in their name.

The variable name contains a space - this is not recommended. Prefer to use underscores or uppercase letters to separate words.

  • The error messages no longer have a link to open the variable editor but I think it's ok because the auto-completion covers the error message anyway.

@D8H
D8H requested a review from 4ian as a code owner September 9, 2026 11:20
@D8H D8H changed the title Fix missing error messages on variable field Fix missing error messages on variable fields Sep 9, 2026
Comment thread newIDE/app/src/EventsSheet/ParameterFields/VariableField.js Outdated
@4ian
4ian merged commit 1149a67 into master Sep 9, 2026
7 checks passed
@D8H
D8H deleted the fix-variable-field-error branch September 9, 2026 16:38
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.

2 participants