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

Asteroids on Parallax #1252

Merged
merged 4 commits into from
Apr 22, 2024
Merged

Asteroids on Parallax #1252

merged 4 commits into from
Apr 22, 2024

Conversation

Kesiath
Copy link
Contributor

@Kesiath Kesiath commented Apr 21, 2024

About the PR

This PR adds a desaturated, darkened, and reduced-amount version of the asteroids parallax background from upstream which should make space around the Frontier feel a bit more dense, and puts it into effect on Frontier.

Why / Balance

While concerns were raised around making sure the background isn't too bright to make it easy to spot EVA people 'skylining' against it, this is hopefully still dark enough to not interfere with that.

Media

image
https://gyazo.com/0e3db03aa3ec0adfd53e2a15725df019

Changelog
🆑

  • tweak: Frontier Station's orbit around Frontier Prime has entered a dense asteroid belt.

@dvir001
Copy link
Contributor

dvir001 commented Apr 21, 2024

Is that the moving one or none moving one?

@Kesiath
Copy link
Contributor Author

Kesiath commented Apr 21, 2024

Is that the moving one or none moving one?

It can move or not move depending on whichever we prefer, currently it'll move, two different layers with the rear one being slower on movement.

Kesiath added 3 commits April 21, 2024 09:50
was used as testing since frontier's parallax wasn't actually set to the correct .yml due to mapping issues
@dvir001
Copy link
Contributor

dvir001 commented Apr 21, 2024

Is that the moving one or none moving one?

It can move or not move depending on whichever we prefer, currently it'll move, two different layers with the rear one being slower on movement.

I was told people get sick by moving background on train in upsteam, this going to be wild.

@Kesiath
Copy link
Contributor Author

Kesiath commented Apr 22, 2024

Is that the moving one or none moving one?

It can move or not move depending on whichever we prefer, currently it'll move, two different layers with the rear one being slower on movement.

I was told people get sick by moving background on train in upsteam, this going to be wild.

Train's movement is incredibly fast, this moves very slow.

@dvir001
Copy link
Contributor

dvir001 commented Apr 22, 2024

Is that the moving one or none moving one?

It can move or not move depending on whichever we prefer, currently it'll move, two different layers with the rear one being slower on movement.

I was told people get sick by moving background on train in upsteam, this going to be wild.

Train's movement is incredibly fast, this moves very slow.

Im willing to give it a test run, worst case we can just stop it from moving.

@dvir001 dvir001 self-requested a review April 22, 2024 19:00
@Kesiath
Copy link
Contributor Author

Kesiath commented Apr 22, 2024

Is that the moving one or none moving one?

It can move or not move depending on whichever we prefer, currently it'll move, two different layers with the rear one being slower on movement.

I was told people get sick by moving background on train in upsteam, this going to be wild.

Train's movement is incredibly fast, this moves very slow.

Im willing to give it a test run, worst case we can just stop it from moving.

https://gyazo.com/0e3db03aa3ec0adfd53e2a15725df019
There's it in action, I'll add this to the PR media as well.

@github-actions github-actions bot added the S: Needs Review This PR is awaiting reviews label Apr 22, 2024
@dvir001 dvir001 merged commit 96a7dc2 into new-frontiers-14:master Apr 22, 2024
11 checks passed
FrontierATC added a commit that referenced this pull request Apr 22, 2024
dvir001 pushed a commit to dvir001/frontier-station-14 that referenced this pull request Jul 10, 2024
It is weird this isn't already a thing, lol.

Signed-off-by: TheOneWhoIsManyFrame <[email protected]>
whatston3 added a commit that referenced this pull request Jul 11, 2024
* The Tiger

* Adds Vulpkanin to the heterochromic eyes marking whitelist (#1434)

Adds Vulpkanin to the heterochromic eyes marking whitelist + ups limit to 3

* Remove Vulpkanin overlay restrictions (#1200)

no longer must you choose two only

* DeltaV species guidebook entries (#1174)

* DeltaV species guidebook entries

Copied straight from the wiki :D

* Forgor

* Update Oni.xml

Signed-off-by: Null <[email protected]>

---------

Signed-off-by: Null <[email protected]>

* Lies

* Alternative wagging tail for felinids. (#593)

* Alternative wagging tail for felinids.

More options for felinids.

Ported from TG station.

* Deletes that one line that appeard for some reason.

---------

Signed-off-by: Adrian16199 <[email protected]>

* Buffed Harpy (#1217)

Now Harpy can mimic more instruments!

Signed-off-by: DefinitelyNotFurryXD <[email protected]>

* Harpy Can Now Mimic Harps! (#1252)

It is weird this isn't already a thing, lol.

Signed-off-by: TheOneWhoIsManyFrame <[email protected]>

* New horns for oni. (#1030)

* New horns for oni.

* Update meta.json

* That one cursed lack of space

* Do what was requested

* Update meta.json

---------

Co-authored-by: Velcroboy <[email protected]>

* ftl real

* Unreal

* Update Dwarf.xml

* Update tattoos.yml

* Update Species.xml

* Update Species.xml

* Missing oni and felinid fluent entries

* Fix missing goblin marking fluent entries

* Fix incorrect goblin marking entries

---------

Signed-off-by: Null <[email protected]>
Signed-off-by: Adrian16199 <[email protected]>
Signed-off-by: DefinitelyNotFurryXD <[email protected]>
Signed-off-by: TheOneWhoIsManyFrame <[email protected]>
Co-authored-by: Adrian16199 <[email protected]>
Co-authored-by: Adeinitas <[email protected]>
Co-authored-by: Tad "Taddy" Johnson <[email protected]>
Co-authored-by: Null <[email protected]>
Co-authored-by: DefinitelyNotFurryXD <[email protected]>
Co-authored-by: TheOneWhoIsManyFrame <[email protected]>
Co-authored-by: Velcroboy <[email protected]>
Co-authored-by: Whatstone <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No C# S: Needs Review This PR is awaiting reviews YML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants