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

add: Skills HUD and Science implants #3937

Closed

Conversation

Mindfri
Copy link
Contributor

@Mindfri Mindfri commented Nov 26, 2023

Описание

Добавляет два новых импланта с Skills HUD'ом и Science. Импланты можно напечатать в протолате и мехфабе. Так же импланты будут лежать в шкафах глав. Science - СМО и РД, Skills HUD - все главы, исключая ГСБ.

Ссылка на предложение/Причина создания ПР

https://discord.com/channels/617003227182792704/755125334097133628/1171850370109157437

Демонстрация изменений

Добавляет два новых импланта для глаз.
@TrashDoxx
Copy link
Contributor

image

Чувак, ну у тебя буквально смежные блоки кода есть для сравнения.

@TrashDoxx
Copy link
Contributor

@TrashDoxx
Copy link
Contributor

Почитай и проблема решится сама собой, как только поймёшь принцип работы.

Да-да
@TrashDoxx
Copy link
Contributor

У тебя по прежнему код не работает так, как нужно. Ты если единичку любому другому импланту пропишешь, оно работать не будет. Ты или класс переменной в цикле смени на родительский, или избавься от переменной scan_items, если функционал твоего объекта должен быть уникальным, потому как она тут лишняя будет.

Copy link
Member

@Daeberdir Daeberdir left a comment

Choose a reason for hiding this comment

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

Про-дол-жа-ем.

code/modules/mob/living/carbon/human/human.dm Outdated Show resolved Hide resolved
@@ -11,6 +11,7 @@
var/see_in_dark = 0
var/see_invisible = 0
var/lighting_alpha
var/scan_items = 0
Copy link
Member

Choose a reason for hiding this comment

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

scan_items стоит заменить на scan_reagents, чтобы не путалось. Ведь это только часть со сканированием реагентов.

@@ -11,6 +11,7 @@
var/see_in_dark = 0
var/see_invisible = 0
var/lighting_alpha
var/scan_items = 0

Copy link
Member

Choose a reason for hiding this comment

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

Далее тебе ещё нужно, чтобы худ видел техи предметов ведь. Это либо нужно actions_types = list(/datum/action/item_action/organ_action/"твой новый экшн") и создать новую штуку, которая будет работать аналогично версии из научных очков, либо insert() remove() проки заставлять выдавать персонажу "видимость технологий" или как её там. Проблема будет в /datum/action/item_action/toggle_research_scanner, который напрочь убирает "видимость технологий" при снятии или переключении очков. Давай-давай, работай дальше.

@Vladisvell Vladisvell added the Won't Fix Автор покинул чат и не желает больше работать label Feb 10, 2024
@endipe endipe mentioned this pull request Aug 5, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Won't Fix Автор покинул чат и не желает больше работать
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants