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

Респрайт некоторых шкафов для локализации #2393

Merged
merged 4 commits into from
Jul 9, 2024

Conversation

chavonadelal
Copy link
Contributor

@chavonadelal chavonadelal commented Jul 8, 2024

Описание PR

Респрайт некоторых шкафов с заменой английских букв на русские, чтобы новички понимали контекст. У СБ буква заменена на иконку для узнаваемости, ранее там тоже была английская буква.
К вопросу почему английская S не была заменена на Б или СБ - лично для меня пиксельная Б слишком похожа на 6, а буквы СБ на ящике выглядят как излишние нагромождение. Ничего же не мешает ящику утилизаторов быть с значком кирки (профессии), а не с буквой. Вот и СБ можно было бы поменять английскую букву на значок профессии.
Медиа

image

Проверки

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

Изменения

🆑 to4no_fix

  • tweak: Респрайт некоторых шкафов для локализации

@chavonadelal chavonadelal requested a review from Morb0 as a code owner July 8, 2024 15:59
@github-actions github-actions bot added 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 labels Jul 8, 2024
Copy link
Contributor

github-actions bot commented Jul 8, 2024

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

Resources/Textures/Structures/Storage/closet.rsi

State Old New Status
cap_door Modified
ce_door Modified
eng_secure_door Modified
hop_door Modified
hos_door Modified
hydro_door Modified
rd_door Modified
science_door Modified
sec_door Modified
warden_door Modified

Edit: diff updated after 13c19f9

@Decortex
Copy link
Contributor

Decortex commented Jul 8, 2024

Мне нравится идея с иконками на шкафу, которые видно в очках сб

@lzk228
Copy link
Collaborator

lzk228 commented Jul 8, 2024

а заменить на букву Б для Безопасности?

@chavonadelal
Copy link
Contributor Author

а заменить на букву Б для Безопасности?

Будет похоже на цифру 6, возможна путаница, я пробовал

@lzk228
Copy link
Collaborator

lzk228 commented Jul 8, 2024

ну Б для Безопасности а 6 то что будет значить?

@lzk228
Copy link
Collaborator

lzk228 commented Jul 8, 2024

image

@lzk228
Copy link
Collaborator

lzk228 commented Jul 8, 2024

image
image
image

@chavonadelal
Copy link
Contributor Author

image

Ну если только так, хотя лично для меня как нагромождение какое-то выглядит.... Морти же пр проверяет? или кто?

@lzk228
Copy link
Collaborator

lzk228 commented Jul 8, 2024

@SonicHDC

@chavonadelal
Copy link
Contributor Author

chavonadelal commented Jul 8, 2024

Ничего же не мешает ящику утилизаторов быть с значком кирки (профессии), а не с буквой. Вот и СБ можно было бы поменять английскую букву на значок профессии.

@chavonadelal
Copy link
Contributor Author

@SonicHDC

Спасибо, подожду ответа

@lzk228 lzk228 requested a review from SonicHDC July 8, 2024 19:14
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.

Честно говоря, у шкафов сб как-то значёк не смотрится. Слишком массивный.
Могу предложить написать на шкафу офицера О, а на шкафу смотрителя С.

@chavonadelal
Copy link
Contributor Author

chavonadelal commented Jul 8, 2024

Честно говоря, у шкафов сб как-то значёк не смотрится. Слишком массивный. Могу предложить написать на шкафу офицера О, а на шкафу смотрителя С.

О и С это кто-то слишком топорно, на мой взгляд. Г и то со скрипом в сердце пришлось нарисовать на ящике ботаника.... Я не понимаю, чем тебе не нравится, ведь на шкафу утилизатора не буква, а кирка - иконка профессии..... Чем же тут хуже? Тебе категорически не нравится? Тем более, что шкафы СБ стилистически отличаются от других.

@chavonadelal
Copy link
Contributor Author

Честно говоря, у шкафов сб как-то значёк не смотрится. Слишком массивный. Могу предложить написать на шкафу офицера О, а на шкафу смотрителя С.

Если совсем не нравится, то могу попробовать нарисовать иконку профессии смотрителя на шкафу смотрителя, чтобы логичнее было.

@SonicHDC
Copy link
Collaborator

SonicHDC commented Jul 9, 2024

Если совсем не нравится, то могу попробовать нарисовать иконку профессии смотрителя на шкафу смотрителя, чтобы логичнее было.

Да. Если сделать так, то будет логично. Тогда спокойно одобрю.

@chavonadelal
Copy link
Contributor Author

chavonadelal commented Jul 9, 2024

Если совсем не нравится, то могу попробовать нарисовать иконку профессии смотрителя на шкафу смотрителя, чтобы логичнее было.

Да. Если сделать так, то будет логично. Тогда спокойно одобрю.

@SonicHDC Перерисовал дверь шкафа смотрителя, теперь там красивая иконка профессии. 😎 Обновил скриншот в Пре, можешь посмотреть. Одобряешь?

@chavonadelal chavonadelal requested a review from SonicHDC July 9, 2024 17:50
@iAmScienceMan
Copy link

Красотулечка, я точно одобряю

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.

Ну теперь это имеет смысл.

@SonicHDC SonicHDC merged commit c463d39 into space-syndicate:master Jul 9, 2024
15 checks passed
@chavonadelal chavonadelal deleted the development branch July 10, 2024 18:01
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.

5 participants