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

Green watery gunk becomes regular water when reflections on #193

Closed
feeltheburn opened this issue May 11, 2022 · 5 comments
Closed

Green watery gunk becomes regular water when reflections on #193

feeltheburn opened this issue May 11, 2022 · 5 comments

Comments

@feeltheburn
Copy link
Contributor

isbroken

As can be seen, this rarely used green watery-type ground tile turns into regular water when reflections are turned on.

feeltheburn added a commit to feeltheburn/Eternal-Lands that referenced this issue May 11, 2022
Fixes greenish watery stuff so it doesn't turn into regular water when reflections are on. Issue raduprv#193
@feeltheburn
Copy link
Contributor Author

feeltheburn commented May 11, 2022

#194

Submitted a pull request that fixes it.

As seen below, the green gunk remains the same with reflections on, just like lava and dirty waters. All water types are displayed, showing the fix does not affect any of them.

(Leonard is also impressed with the results.)

isfixed

@gvissers
Copy link
Collaborator

If I'm seeing the first picture correctly, it doesn't turn into a regular water tile, but a slightly greenish water (compare it with regular water in the lower right). I will admit that I had expected the effect of the water texture to be stronger, though, also for the tile under the lava. Fix looks fine to me, if you are happy with water tiles 233-239 not reflecting.

@feeltheburn
Copy link
Contributor Author

feeltheburn commented May 11, 2022

234-239 do not exist, and pretty much zero chance they ever will with there being no use for them as the maps are at a pretty much set state.

If a new reflective water gets added, testing it will show this file needs to change with it, and it would be better to number it in the 221-229 fields, changing #define IS_WATER_TILE(i) (!i || (i>230 && i<255)) to i>220.

There is a very, very slight difference in water color with the current version, but this water is obviously meant to be more like some kind of... I wanna say radioactive gunk? But it does seem it should behave more like the dirty water, with no change.

@feeltheburn
Copy link
Contributor Author

feeltheburn commented May 11, 2022

This is the lower end of the tiles as shown in the map editor. Unseen "0" at the very top is the most commonly seen water, then there's the 6 "water" tiles seen here in the 200s. All 7 total are seen in the above images.

As the white areas show, there's a wealth of unused numbers.

tiles

pjbroad pushed a commit that referenced this issue Nov 20, 2022
Fixes greenish watery stuff so it doesn't turn into regular water when reflections are on. Issue #193
@feeltheburn
Copy link
Contributor Author

Closing as the commit solves the issue.

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

No branches or pull requests

2 participants