Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use height_3d also for background sprite, not only for foreground #60994

Merged
merged 1 commit into from Sep 14, 2022

Conversation

mlange-42
Copy link
Contributor

Summary

Interface "Use height_3d also for background sprite, not only for foreground"

Purpose of change

Render things like shadows on the same hight as the foreground sprite.

Describe the solution

Apply the same height for the background that is used for the foreground, but don't stack onto the height for subsequent draws.

Describe alternatives you've considered

Leave as is.

Testing

Tried in game (see screenshot)

Additional context

Screenshot with player shadow on the correct height:

grafik

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Sep 14, 2022
@NetSysFire NetSysFire added the SDL: Tiles / Sound Tiles visual interface and sounds. label Sep 14, 2022
@dseguin dseguin merged commit 1edc0a5 into CleverRaven:master Sep 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. json-styled JSON lint passed, label assigned by github actions SDL: Tiles / Sound Tiles visual interface and sounds.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants