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

New wooden tiles + Resprite old #2243

Closed
wants to merge 8 commits into from

Conversation

NotSoDana
Copy link
Contributor

@NotSoDana NotSoDana commented Jun 12, 2024

Описание PR

Добавил много-много деревянных тайлов разного вида и цвета для маппинга. Также завёз респрайт старых деревяшек ибо они фу. Пока не добавляю крафт, ибо лень, как будет время завезу.
Почему не оффам? Маппинг их волнует меньше всего + феатур фриз, на Корваксе намного больше толку будет от этого всего.

Медиа

Screenshot_35
Screenshot_36

Проверки

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

Изменения

🆑 Dezzzix, Meowstushka

  • add: Добавлены тёмные, красные, чёрные и белые деревянные тайлы разных видов.
  • add: Добавлен деревянный паркет
  • tweak: Респрайт деревянных тайлов

@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 Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation labels Jun 12, 2024
Copy link
Contributor

github-actions bot commented Jun 12, 2024

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

Resources/Textures/Corvax/Objects/Tiles/wood.rsi

State Old New Status
wood-black-inhand-left Added
wood-black-inhand-right Added
wood-dark-inhand-left Added
wood-dark-inhand-right Added
wood-light-inhand-left Added
wood-light-inhand-right Added
wood-red-inhand-left Added
wood-red-inhand-right Added
wood_black Added
wood_chess Added
wood_chess_black Added
wood_chess_dark Added
wood_chess_light Added
wood_chess_red Added
wood_dark Added
wood_large_black Added
wood_large_dark Added
wood_large_light Added
wood_large_red Added
wood_light Added
wood_parquet Added
wood_parquet_black Added
wood_parquet_dark Added
wood_parquet_light Added
wood_parquet_red Added
wood_red Added

Edit: diff updated after 13f456c

@MureixloI
Copy link
Contributor

Может палитру деревянного пола основного поменять на палитру деревянной мебели?

@SonicHDC
Copy link
Collaborator

У меня есть 2 претензии:

  1. Зачем заменять старый узор, если он хорошо смотрится?
  2. У нас плитка с освещением с левого верхего угла, а не снизу.
    2.1) Плитка слишком объёмная, от части выбивается от остальных тайлов и отвлекает.
    2.2) Мы не стримимся к новому ТГ и не собираемся.
    Советую модифицировать данные спрайты под стиль стального пола или создать их с нуля.

@NotSoDana
Copy link
Contributor Author

NotSoDana commented Jun 12, 2024

  1. Зачем заменять старый узор, если он хорошо смотрится?

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

@Ko4ergaPunk
Copy link
Collaborator

Прифки. У светлого и красного дерева явный перебор с насыщенностью, а у светлого еще и с контрастностью. Он прям какой-то грязнущий...
Сотри еще, пожалуйста, вот эти баребухи крестообразные. Из-за них очень уж бросается в глаза паттерн
image

@lzk228
Copy link
Collaborator

lzk228 commented Jun 13, 2024

оффам

@BolloTea
Copy link

Мы не стримимся к новому ТГ и не собираемся.
Советую модифицировать данные спрайты под стиль стального пола или создать их с нуля.
Тем временем большинство спрайтов одежды и хардсьютов взяты с тг

@NotSoDana
Copy link
Contributor Author

NotSoDana commented Jun 13, 2024

resprited :Cat_blehhh:

@lzk228
Copy link
Collaborator

lzk228 commented Jun 13, 2024

оффам

@Deserty0
Copy link
Contributor

оффам :trollface:

@SonicHDC
Copy link
Collaborator

SonicHDC commented Jun 13, 2024

Прежде чем добавлять кучу видов и узоров дерева, лучше их сначала добавить оффам.
Фриз кончился, можно пилить.
А уже после респрайтить.
Увы.

@NotSoDana
Copy link
Contributor Author

Я видимо прикола ради строчку "почему не оффам" написал

@NotSoDana
Copy link
Contributor Author

ладно, пока так, позже глянем что к чему

@NotSoDana NotSoDana closed this Jun 13, 2024
@NotSoDana
Copy link
Contributor Author

NotSoDana commented Jun 14, 2024

Ну, о чём собственно и шла речь. #28963
Я не остановлюсь, мапперам корвуха это всё дело действительно не помешает. При том с моим мнением даже у оффов есть согласные, просто по мнению Эмо такая вариативность им лишняя.
image
image

@NotSoDana NotSoDana reopened this Jun 14, 2024
@Ko4ergaPunk
Copy link
Collaborator

Она не остановится

@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label Jun 18, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@NotSoDana
Copy link
Contributor Author

bruh

@github-actions github-actions bot removed the Merge Conflict PR have unsolved merge conflicts label Jun 21, 2024
@github-actions github-actions bot added the Merge Conflict PR have unsolved merge conflicts label Jun 21, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@NotSoDana
Copy link
Contributor Author

да ебись ты

@NotSoDana
Copy link
Contributor Author

Ясно, соурс три дал ёбу. Сделаем проще..

@NotSoDana NotSoDana closed this Jun 21, 2024
@NotSoDana NotSoDana deleted the New-wood-tiles branch June 21, 2024 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Can be reviewed or fixed by people who are knowledgeable with translation 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 Merge Conflict PR have unsolved merge conflicts 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