Skip to content

Conversation

@wjt
Copy link
Member

@wjt wjt commented Feb 11, 2026

For all scenes in the NO_EDIT template, change the root node name to be NoEdit followed by the quest name, matching the filename.

When instantiating the template, rewrite the root node name to replace NoEdit with the PascalCase name of the quest, so that, again, the root node matches the filename.

Resolves #1911

@wjt wjt requested review from a team as code owners February 11, 2026 10:45
@wjt wjt marked this pull request as draft February 11, 2026 10:45
@wjt

This comment was marked as outdated.

@wjt wjt force-pushed the wjt/prefix-template-scene-root-nodes-with-quest-name branch from 5074d74 to 98e141a Compare February 11, 2026 10:46
@github-actions
Copy link

Play this branch at https://play.threadbare.game/branches/endlessm/wjt/prefix-template-scene-root-nodes-with-quest-name.

(This launches the game from the start, not directly at the change(s) in this pull request.)

Copy link
Collaborator

@manuq manuq left a comment

Choose a reason for hiding this comment

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

Yes, great idea.

Comment on lines +106 to +107
var pascal_case_name := quest_name.capitalize().replace(" ", "")
scene.name = pascal_case_name + suffix
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

For all scenes in the NO_EDIT template, change the root node name to be
NoEdit followed by the quest name, matching the filename.

When instantiating the template, rewrite the root node name to replace
NoEdit with the PascalCase name of the quest, so that, again, the root
node matches the filename.

Resolves #1911
@wjt wjt force-pushed the wjt/prefix-template-scene-root-nodes-with-quest-name branch from 98e141a to 0d82761 Compare February 11, 2026 12:07
@wjt wjt removed request for a team February 11, 2026 12:08
@wjt wjt marked this pull request as ready for review February 11, 2026 12:08
@wjt wjt merged commit 2f926e7 into main Feb 11, 2026
6 checks passed
@wjt wjt deleted the wjt/prefix-template-scene-root-nodes-with-quest-name branch February 11, 2026 12:43
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.

Prefix NO_EDIT root nodes with quest name

2 participants