Skip to content

Revert "feat: Fancy Screenshake from Ephemeral Space (#5503)"#5955

Open
keekee38 wants to merge 1 commit into
DeltaV-Station:masterfrom
keekee38:revert-5503-screenshake
Open

Revert "feat: Fancy Screenshake from Ephemeral Space (#5503)"#5955
keekee38 wants to merge 1 commit into
DeltaV-Station:masterfrom
keekee38:revert-5503-screenshake

Conversation

@keekee38
Copy link
Copy Markdown
Contributor

@keekee38 keekee38 commented Jun 3, 2026

About the PR

reverting #5503

Why / Balance

it kinda gave me motion sickness and think some other folks were sayin they werent fans of it either

Technical details

Media

its just the old screenshake

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an in-game showcase.

Licensing

Changelog

🆑

  • tweak: Changed screenshake back to how it was before

@keekee38 keekee38 requested review from a team and Toby222 as code owners June 3, 2026 01:17
@github-actions github-actions Bot added size/L 256-1023 lines S: Needs Review Awaiting review from a Maintainer Changes: YML Changes any yml files Changes: UI Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: Documentation Changes any xml or md files labels Jun 3, 2026
@delta-v-bot delta-v-bot Bot added the Under Direction Review This pull request is under consideration from the Direction team. label Jun 3, 2026
@delta-v-bot
Copy link
Copy Markdown

delta-v-bot Bot commented Jun 3, 2026

Triaged by starboundsolaris:
This PR requires a content review discussion, which will be held in private.

DIRECTION KNOWS BEST
DIRECTION KNOWS BEST
DIRECTION KNOWS BEST

The review duration has been set to 7 days.

@turtlemutt
Copy link
Copy Markdown
Contributor

thank god

@pootslap
Copy link
Copy Markdown
Contributor

pootslap commented Jun 3, 2026

I'm so glad it's not just me. The new screenshake is kinda cool for when stuff hits the station but fighting someone with a knife and causing a huge earthquake every time you swing looks bizarre.

@BarryNorfolk
Copy link
Copy Markdown
Contributor

This has a dedicated option in settings, I think?

@Axianamos
Copy link
Copy Markdown

It does have an option, which i immediately disabled. I dont think the feature should be removed, but perhaps it would be better to start with it disabled.

@keekee38
Copy link
Copy Markdown
Contributor Author

keekee38 commented Jun 3, 2026

mhmm theres an option in settings but it just disables screenshake entirely, i like screenshake but this one i cant use at all

@DisposableCrewmember42
Copy link
Copy Markdown
Contributor

DisposableCrewmember42 commented Jun 4, 2026

Some changes that could be made instead of removing completely:

  • It would be possible to use the old screenshake for melee attacks, which I agree looks weird right now.
  • We could combine the old screenshake with the new screenshake for explosions so there's some directionality to it again. Starlight has that as a tweak to it, could be ported.
  • The severity of the shaking can be tweaked

I do think this new screenshake is far better for shooting than the old one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: C# Changes any cs files Changes: Documentation Changes any xml or md files Changes: Localization Changes any ftl files Changes: UI Changes: YML Changes any yml files S: Needs Review Awaiting review from a Maintainer size/L 256-1023 lines Under Direction Review This pull request is under consideration from the Direction team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants