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

[Art] Mining structures retexture #2465

Merged
merged 10 commits into from
Aug 14, 2024

Conversation

Ko4ergaPunk
Copy link
Collaborator

Описание PR

Перерисовал шахтерские стены, окна и плитки. Оригинал очень неплох, но по цветам с корвухом не сходится, да и по форме.

От порушенных версий решил отказаться, так как нахер они и не нужны =)
Но могу сделать, если прям так хочется. Правда, я даже не знал о их существовании. Ну и убого они выглядят, есчесн.

Медиа

pr

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

🆑

…об этом в коммите. Йоу, шахтеры! Идите нахуй шахтёры B-) Я лютый старатель. Копайте свои ебучие шахты, пока я в луже золото добываю. СОСАТЬ ШАХТЕРЫ!! B-) Эй, ты шахтер? Значит ты ЧМООООООООООООООООООООООООООООООООООООООООООООООООООООООООО!!!!!!!!! Ахахахахаха ЧМОООООООООООО
@Ko4ergaPunk Ko4ergaPunk requested a review from Morb0 as a code owner August 7, 2024 16:16
@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design and removed Status: Needs Review This PR requires new reviews before it can be merged labels Aug 7, 2024
@Ko4ergaPunk
Copy link
Collaborator Author

@SonicHDC

Copy link
Contributor

github-actions bot commented Aug 7, 2024

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

Resources/Textures/Objects/Tiles/tile.rsi

State Old New Status
mining Modified
miningdark Modified
mininglight Modified

Resources/Textures/Structures/Doors/Airlocks/Glass/mining.rsi

State Old New Status
assembly Modified
closed Modified
closing Modified
open Modified
opening Modified
panel_closing Modified
panel_open Modified
panel_opening Modified

Resources/Textures/Structures/Doors/Airlocks/Standard/mining.rsi

State Old New Status
assembly Modified
closed Modified
closing Modified
open Modified
opening Modified
panel_closing Modified
panel_open Modified
panel_opening Modified

Resources/Textures/Structures/Walls/mining.rsi

State Old New Status
full Modified
mining0 Modified
mining1 Modified
mining2 Modified
mining3 Modified
mining4 Modified
mining5 Modified
mining6 Modified
mining7 Modified
miningB0 Modified
miningB1 Modified
miningB2 Modified
miningB3 Modified
miningB4 Modified
miningB5 Modified
miningB6 Modified
miningB7 Modified

Resources/Textures/Structures/Walls/mining_diagonal.rsi

State Old New Status
state0 Modified
state1 Modified

Resources/Textures/Structures/Windows/mining.rsi

State Old New Status
full Modified
mwindow0 Modified
mwindow1 Modified
mwindow2 Modified
mwindow3 Modified
mwindow4 Modified
mwindow5 Modified
mwindow6 Modified
mwindow7 Modified

Resources/Textures/Structures/Windows/mining_diagonal.rsi

State Old New Status
state0 Modified
state1 Modified

Edit: diff updated after ee1903b

lzk228
lzk228 previously requested changes Aug 7, 2024
Copy link
Collaborator

@lzk228 lzk228 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

От порушенных версий решил отказаться, так как нахер они и не нужны =)

либо делаешь нормально либо никак

@lzk228 lzk228 added the Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged label Aug 7, 2024
@Sergan31
Copy link

Sergan31 commented Aug 7, 2024

то есть стены не сочетались по стилю ровно до момента, пока их не сделали более серыми?

Copy link
Collaborator

@SonicHDC SonicHDC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Окна слишком светлые, лучше приблизить к матовому варианту.

@poeMota
Copy link
Contributor

poeMota commented Aug 7, 2024

окна кайф, предыдущие невозможно было использовать

@BolloTea
Copy link

BolloTea commented Aug 7, 2024

Фу, светлая тема...

@NotSoDana
Copy link
Contributor

Чет по-моему хуета какая-то, особенно стекло

@BolloTea
Copy link

BolloTea commented Aug 8, 2024

Мрачность и темное стекло лучше шли. Теперь это палатка для кемпинга, а не суровых шахтеров

@Ko4ergaPunk
Copy link
Collaborator Author

Взял цвета с матового
@SonicHDC
pr

@Ko4ergaPunk Ko4ergaPunk requested a review from SonicHDC August 8, 2024 04:16
@github-actions github-actions bot added Status: Needs Review This PR requires new reviews before it can be merged and removed Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged labels Aug 8, 2024
@Ko4ergaPunk
Copy link
Collaborator Author

image
В игре показать сложно, тк они рандомные

@BolloTea
Copy link

BolloTea commented Aug 8, 2024

Матовые стекла хорошо выглядят. Кочерга, ты же делал респрайт чтобы они более к ванильной стене подходили?

@Ko4ergaPunk
Copy link
Collaborator Author

Матовые стекла хорошо выглядят. Кочерга, ты же делал респрайт чтобы они более к ванильной стене подходили?

Не. Я матовые не трогал

@Ko4ergaPunk
Copy link
Collaborator Author

ПОБЕДА @SonicHDC

SonicHDC
SonicHDC previously approved these changes Aug 9, 2024
Copy link
Collaborator

@SonicHDC SonicHDC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Всё хорошо, под вечер сам мёрджну.
Вдруг кто раньше продискутирует или mergeнёт.

@lzk228
Copy link
Collaborator

lzk228 commented Aug 9, 2024

image

мне кажется сломанные стены не достаточно сломанные по сравнению с предыдущими
да и обычные слишком гладкие а именно просто полоска одним цветом, когда предыдущие были более потёртые

@Ko4ergaPunk
Copy link
Collaborator Author

image

мне кажется сломанные стены не достаточно сломанные по сравнению с предыдущими да и обычные слишком гладкие а именно просто полоска одним цветом, когда предыдущие были более потёртые

Полоску лучше не трогать. Выглядит крайне паршиво, ибо паттерн слишком сильно в глаза бросается

@SonicHDC
Copy link
Collaborator

SonicHDC commented Aug 11, 2024

image
мне кажется сломанные стены не достаточно сломанные по сравнению с предыдущими да и обычные слишком гладкие а именно просто полоска одним цветом, когда предыдущие были более потёртые

Полоску лучше не трогать. Выглядит крайне паршиво, ибо паттерн слишком сильно в глаза бросается

Будь проще, сделай сломаный вариант в прототипе стенок. Хитро и смешно :trollface:

@Ko4ergaPunk
Copy link
Collaborator Author

Добавил черкаши на полоску
image
@SonicHDC

@SonicHDC SonicHDC dismissed lzk228’s stale review August 14, 2024 10:24

Стенку сломали, неактульно

@SonicHDC SonicHDC merged commit 061f67d into space-syndicate:master Aug 14, 2024
12 of 13 checks passed
@Ko4ergaPunk Ko4ergaPunk deleted the mining-walls branch August 14, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Code has not been changed Changes: Sprites Should be reviewed or fixed by people who are knowledgeable with spriting or visual design Status: Needs Review This PR requires new reviews before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants