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

Items in transparent pockets on the map illuminate surroundings #78789

Merged
merged 1 commit into from
Dec 27, 2024

Conversation

mqrause
Copy link
Contributor

@mqrause mqrause commented Dec 27, 2024

Summary

Bugfixes "Items in transparent pockets on the map illuminate surroundings"

Purpose of change

Currently only top level items on the map are considered as light sources. This excludes for example candles in candle sticks.

Describe the solution

Map items also check contents of transparent pockets for light sources.
Straight up cherry-picked from #67595, only fixed conflicts from typed tripoints.

Describe alternatives you've considered

Do the work again and still end up with the same solution?

Testing

Put candle in candlestick and place on floor.

Additional context

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Dec 27, 2024
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Dec 27, 2024
@Maleclypse Maleclypse merged commit a014c17 into CleverRaven:master Dec 27, 2024
24 of 27 checks passed
@mqrause mqrause deleted the map_pocket_lights branch December 27, 2024 17:56
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 <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants