Skip to content

Signpost: Add non-directional variant #2933 - #2936

Merged
manuq merged 5 commits into
endlessm:mainfrom
joshuajenner:Add-non-directional-sign-variant
Sep 28, 2026
Merged

manuq merged 5 commits into
endlessm:mainfrom
joshuajenner:Add-non-directional-sign-variant

Conversation

@joshuajenner

Copy link
Copy Markdown
Contributor

Updated the sign sprite to include a non-directional variant.
Updated the update_appearance() function in the sign scene to use the correct frame when setting the direction variable.

Resolves: #2933

@joshuajenner
joshuajenner requested review from a team as code owners September 20, 2026 10:29

@manuq manuq left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is looks really promising! It works, I'm only requesting code improvements.

Comment thread assets/third_party/tiny-swords/Deco/17.png
Comment thread scenes/game_elements/props/sign/components/sign.gd Outdated
Comment thread scenes/game_elements/props/sign/components/sign.gd Outdated
@github-actions

github-actions Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Test build no longer available.

@joshuajenner
joshuajenner requested a review from a team as a code owner September 24, 2026 16:30
@joshuajenner

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback.

Undid the edit to the third party sprite, and moved the edited sprite to the non-cc0 folder.
Added a Frame enum.
Set the Appearance sprite to use unique name.

@joshuajenner
joshuajenner requested a review from manuq September 26, 2026 07:44
@manuq

manuq commented Sep 28, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the feedback.

Undid the edit to the third party sprite, and moved the edited sprite to the non-cc0 folder.

Sorry for confusing you in #2936 (comment) . As you can see, English is not my native language :) I wanted to say that the change was fine. For context:

  • PixelFrog is not going to update the CC0 asset pack, so I think is fine changing it as you did.
  • PixelFrog gave us permission to ship the non-CC0 asset pack as-is. If they ever want to update the pack we should ask them for permission again.

Don't worry, I will revert the commit and merge, since I caused the confusion.

Added a Frame enum. Set the Appearance sprite to use unique name.

Looks good. Thanks!

This StoryQuest had the scene duplicated.
@manuq

manuq commented Sep 28, 2026

Copy link
Copy Markdown
Collaborator

The other signpost scene changed here was a duplicate in a StoryQuest. Since there weren't any customizations, I went ahead and replaced the single instantiation by the main one, and removed it.

@manuq
manuq merged commit f654a7e into endlessm:main Sep 28, 2026
6 checks passed
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.

Signpost: Add non-directional variant

2 participants