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

Add submerge to oasis terrain #8812

Merged
merged 1 commit into from Apr 28, 2024
Merged

Conversation

Jonathan-Kelly
Copy link
Contributor

Follow up to #8783 this makes a unit on an oasis tile visibly sink slightly into its waters, just like with ford terrain.

Follow up to wesnoth#8783 this makes a unit on an oasis tile visibly sink slightly into its waters, just like with ford terrain.
@github-actions github-actions bot added the Terrain Issues that involve terrain definitions or their implementation in the engine. label Apr 27, 2024
@knyghtmare
Copy link
Member

I really think the oasis terrain tile can be improved in appearance to better accompany the terrain type change (I saw it was shallow water and desert now) and also the submerge visual effect.

@soliton-
Copy link
Member

It's only shallow water. Desert is just the default base as before.

@stevecotton
Copy link
Contributor

Looks good to me, but I guess @doofus-01 should approve this.

@doofus-01
Copy link
Member

Given that oasis is now (sort of) shallow water, and shallow water is 0.4 and ford is 0.3, 0.3 seems reasonable.

Maybe 0.2 would work too, but I haven't tested either one.

@stevecotton stevecotton merged commit 890caf5 into wesnoth:master Apr 28, 2024
19 checks passed
@Jonathan-Kelly Jonathan-Kelly deleted the oasis branch May 2, 2024 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Terrain Issues that involve terrain definitions or their implementation in the engine.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants