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

fix(UI): Added missing sprite definitions for overmap auto travel path #5565

Merged

Conversation

thedyze
Copy link
Contributor

@thedyze thedyze commented Oct 12, 2024

Checklist

Required

Optional

Purpose of change

Fix fallback sprite showing for the auto travel path on the overmap. This was because of missing definitions in the UDP tile_config.

Describe the solution

375359762-d7a16899-cbc9-450b-a17b-9d473d800670

Also assigned a semitransparent sprite for when the path is on a different z-level than currently viewed. (id's: "overmap_path_above" and "overmap_path_below")

image

Describe alternatives you've considered

Testing

Additional context

Copy link
Member

@chaosvolt chaosvolt left a comment

Choose a reason for hiding this comment

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

:D

@chaosvolt chaosvolt merged commit 1b7a922 into cataclysmbnteam:main Oct 12, 2024
7 checks passed
@scarf005
Copy link
Member

scarf005 commented Nov 1, 2024

related: #5556

@thedyze thedyze deleted the overmapPathTileIdsMissingFix branch November 3, 2024 10:51
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.

3 participants