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.

Sign up for GitHub

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

Jump to bottom

Gold & Silver wall resprite #32099

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Gold & Silver wall resprite #32099

wants to merge 4 commits into from
+92 −2

Conversation

TheShuEd
Copy link
Member

About the PR

Resprited to the same style as default solid walls

Media

Before:
image
image

After:
image
image

@github-actions github-actions bot added No C# For things that don't need code. Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design. labels Sep 12, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 12, 2024 •
edited
Loading

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

Resources/Textures/Structures/Walls/gold.rsi

State Old New Status
full Modified
gold0 Modified
gold1 Modified
gold2 Modified
gold3 Modified
gold4 Modified
gold5 Modified
gold6 Modified
gold7 Modified

Resources/Textures/Structures/Walls/silver.rsi

State Old New Status
full Modified
silver0 Modified
silver1 Modified
silver2 Modified
silver3 Modified
silver4 Modified
silver5 Modified
silver6 Modified
silver7 Modified

Edit: diff updated after 12d6f19

@github-actions github-actions bot added the Status: Needs Review This PR requires new reviews before it can be merged. label Sep 12, 2024
@BolloTea
Copy link

image

@Everturning
Copy link

Everturning commented Sep 12, 2024 •
edited
Loading

Gold looks good but silver doesn't look silver to me. It looks more like steel to me. Can you make it brighter in the center?

@TheShuEd TheShuEd mentioned this pull request Sep 13, 2024
Open
16 tasks
@Emisse
Copy link
Contributor

Emisse commented Sep 16, 2024

Border of the walls is too bright

@whateverusername0
Copy link
Contributor

the butter room

@TheShuEd
Copy link
Member Author

image
image
fixed 2 months ago

@TheShuEd
Copy link
Member Author

image
image

@TheShuEd
Copy link
Member Author

Flareguy feedback applied

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

@Flareguy Flareguy Awaiting requested review from Flareguy

@EmoGarbage404 EmoGarbage404 Awaiting requested review from EmoGarbage404

Assignees
No one assigned
Labels
Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design. No C# For things that don't need code. Status: Needs Review This PR requires new reviews before it can be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@TheShuEd @BolloTea @Everturning @Emisse @whateverusername0