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

Ref: Dir standartization (or not) #276

Closed
wants to merge 14 commits into from
Closed

Ref: Dir standartization (or not) #276

wants to merge 14 commits into from

Conversation

AyIong
Copy link
Collaborator

@AyIong AyIong commented Oct 3, 2023

Что этот PR делает

Ох зря я сюда полез.
Стандартизация направлений настенной машинерии, перенос направлений в отдельный объект, меньше лишних VV в картах...

Почему это хорошо для игры

Чуть меньше путанницы с направлениями, и никаких пожарных сигнализаций, подписанных как "north bump"

Изображения изменений

Не должно бросаться в глаза

Тестирование

Ох бля...

Changelog

🆑
spellcheck: Убрал лишние префабы направлений, не должно быть заметное в игре.
/:cl:

@github-actions github-actions bot added 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation) Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) labels Oct 3, 2023
@@ -0,0 +1,518 @@
/* Intercom */
/obj/item/radio/intercom
Copy link

Choose a reason for hiding this comment

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

А зачем базовые объекты тут?

/obj/item/radio/intercom
name = "\improper station intercom"

/obj/item/radio/intercom/directional
Copy link

Choose a reason for hiding this comment

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

И такие

dir = 8

/* WallMed */
/obj/machinery/economy/vending/wallmed/directional
Copy link

Choose a reason for hiding this comment

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

И там далее, как я понимаю тоже будет вот такое

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Базовые объекты шоб переименовать, спорно наверное но да ладно
directional шоб не было условного интеркома с именем north, и не путалсь с другими вещами

@github-actions github-actions bot removed the 🖌️ Спрайты Вы заработали свою миска-рис и кошко-жена. Партия гордится вами! label Oct 3, 2023
@github-actions github-actions bot added the 🎸 Инструменты Мы выдаем себя за реальное сообщество разработчиков. label Oct 3, 2023
@AyIong
Copy link
Collaborator Author

AyIong commented Oct 3, 2023

Закрыто на переработку, всё слишком не красиво

@AyIong AyIong closed this Oct 3, 2023
@AyIong AyIong deleted the maps-cleanup branch November 6, 2023 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎸 Инструменты Мы выдаем себя за реальное сообщество разработчиков. 🗺️ Изменение Карты В этом ПРе затронут файл не станционной карты. Может и не один. Керберос Этот ПР изменил что-то на Керберосе (DeltaStation) Кибериада Этот ПР изменил что-то на Кибериаде (BoxStation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants