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

Remove sky from snow backgrounds #1801

Merged
merged 2 commits into from
Aug 13, 2021
Merged

Remove sky from snow backgrounds #1801

merged 2 commits into from
Aug 13, 2021

Conversation

Rusty-Box
Copy link
Member

The affected snow backgrounds are: arctis2.png and arcticmountain.png

I still kept the not transparent version of arctis2 to be used as the background for the level editor's start screen without the black void visible. It is located separately from the other backgrounds in images/engine/menu/ to not cause confusion.

Affected backgrounds: arctis2.png and arcticmountain.png
Use version of arctis2 background with sky for the level editor start screen.
@Rusty-Box Rusty-Box added involves:editor category:design status:needs-review Work needs to be reviewed by other people labels Aug 2, 2021
@serano01
Copy link
Member

serano01 commented Aug 2, 2021

I'm not against it, but what's the occasion? Plus, after merging, someone needs to add a gradient for the levels which are using them, otherwise they will have a black void visible.

@Rusty-Box
Copy link
Member Author

@serano01 This allows these backgrounds to be layered and combined together. This is also beneficial for having them set at different times of day or varing sky colors in general.

Also, I personally see them having much more of a future then parallax tiles (atleast the way we use them right now).

@serano01 serano01 merged commit 940cf40 into SuperTux:master Aug 13, 2021
@mrkubax10 mrkubax10 removed the status:needs-review Work needs to be reviewed by other people label Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants