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

Minor diagonal decal adjustments #669

Merged
merged 1 commit into from
Dec 10, 2023
Merged

Minor diagonal decal adjustments #669

merged 1 commit into from
Dec 10, 2023

Conversation

Blackszedows
Copy link
Contributor

About the PR

Adjusted diagonal decals i made a while ago (still not perfect).
Mainly:

  • made the half-tile diagonal a bit wider to fit regular half-tile decal more (see pictures)
  • same treatment to corner pieces (now they stick out a pixel or two when using them at corner of perpendicular decals , hence the "not perfect, but i didn't want to make 8-16 different decals that differ by a pixel or two)

Why / Balance

I'm not good at making sprites by a mile, hence figured i should fix what i can, if someone else wants to make modifications feel free to.

Media

Diagonal pieces 2x2 (the corner pieces are also there)
Before:
Before
After:
After
Preview with a wall:
Preview

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

🆑

  • tweak: Adjusted the sprites for diagonal tile decals.

Copy link
Contributor

github-actions bot commented Dec 7, 2023

RSI Diff Bot; head commit e3b639d merging into 956cbe8
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Decals/Overlays/greyscalediagonal.rsi

State Old New Status
tilediagonal Modified
tilediagonal_corner Modified
tilediagonal_corner_180 Modified
tilediagonal_corner_270 Modified
tilediagonal_corner_90 Modified
tilediagonal_flipped Modified

@Blackszedows
Copy link
Contributor Author

To note, i did check with a ship of mine that had previous decals , and there shouldn't be any conflicts. Shouldn't.

@Cheackraze Cheackraze self-requested a review December 10, 2023 17:54
@github-actions github-actions bot added the Status: Needs Review This PR is awaiting reviews label Dec 10, 2023
@Cheackraze Cheackraze merged commit 996647f into new-frontiers-14:master Dec 10, 2023
11 checks passed
FrontierATC added a commit that referenced this pull request Dec 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants