Add cliff shadows to quest_001 - 2_ink_combat

[russell-cj]

Base shadows have been added to the three ink combat scenes in Quest 001. A new TileMapLayer called Shadows was added to each scene, using the tiles/shadows.tres tileset. Shadows were painted at all contact points between cliffs and the ground.

Resolves #1896

[github-actions]

Play this branch at https://play.threadbare.game/branches/endlessm/issue_1896.

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

[wjt]

Shadows were painted at all contact points between cliffs and the ground.

Emphasis mine. This is exactly right! Consider these cliffs:

You have correctly painted a row of four shadow tiles where I have drawn a purple box - which is where the cliffs meet the ground. You have correctly not painted shadow tiles where I have drawn an orange box. This makes the fake perspective look correct: we are looking down on the scene at an angle, and so we can't see the shadows that are "behind" the cliffs from our point of view.

This took me a while to get my head around for foam - same thing:

I wanted to paint foam at the top edge of raised islands like this, until someone explained the perspective point to me.

[wjt]

One thing I was not sure about at first is the layering of shadows on top of foam:

Screencast.From.2026-02-10.10-31-25.mp4

This is conceptually correct: the cliff must cast a shadow onto the water, and it must be on top of the foam. It's also unavoidable with the way we have the shadow tile set up, because we definitely want the shadow to the right of that cliff where it meets the grass.

I think it looks a little weird but I think it's because I'm not used to it.

[wjt]

For future reference you don't need to explicitly request review - the .github/CODEOWNERS file in the repository has rules that tell GitHub which teams to tag for review, and Manuel and I are in the Games team that is the owner of these files.

[wjt]

I've edited the description to remove these sections:

Modified scenes

• ink_combat_round_1.tscn
• ink_combat_round_2.tscn
• ink_combat_round_3.tscn

This isn't necessary - Git tells us which files are modified.

🎮 Play 2_ink_combat

Play 2_ink_combat

This is great for testing but the PR message becomes the commit message, and we don't want this link in the permanent commit history because the link will stop working as soon as the PR is merged. Perhaps in future it could be a comment?

[russell-cj]

Thanks, Will! Everything is very clear now. With these guidelines in mind, executing the remaining sub-tasks will be much more fluid.